Update stale code comment in CheckpointerMain()
Hi,
The attached patch updates the code comment which is no longer true
after commit # 4a92a1c3d1c361ffb031ed05bf65b801241d7cdd
--
Regards,
Amul Sul
EDB: http://www.enterprisedb.com
Attachments:
update_code_comment.patchapplication/x-patch; name=update_code_comment.patchDownload+1-5
On 30 Nov 2021, at 08:00, Amul Sul <sulamul@gmail.com> wrote:
The attached patch updates the code comment which is no longer true
after commit # 4a92a1c3d1c361ffb031ed05bf65b801241d7cdd
Agreed, but looking at this shouldn't we also tweak the comment on
RecoveryInProgress() as per the attached v2 diff?
--
Daniel Gustafsson https://vmware.com/
Attachments:
update_code_comment_v2.patchapplication/octet-stream; name=update_code_comment_v2.patch; x-unix-mode=0644Download+3-7
On Tue, Nov 30, 2021 at 3:09 PM Daniel Gustafsson <daniel@yesql.se> wrote:
On 30 Nov 2021, at 08:00, Amul Sul <sulamul@gmail.com> wrote:
The attached patch updates the code comment which is no longer true
after commit # 4a92a1c3d1c361ffb031ed05bf65b801241d7cddAgreed, but looking at this shouldn't we also tweak the comment on
RecoveryInProgress() as per the attached v2 diff?
Yes, we should -- diff looks good to me, thanks.
Regards,
Amul
On 1 Dec 2021, at 07:19, Amul Sul <sulamul@gmail.com> wrote:
On Tue, Nov 30, 2021 at 3:09 PM Daniel Gustafsson <daniel@yesql.se> wrote:
On 30 Nov 2021, at 08:00, Amul Sul <sulamul@gmail.com> wrote:
The attached patch updates the code comment which is no longer true
after commit # 4a92a1c3d1c361ffb031ed05bf65b801241d7cddAgreed, but looking at this shouldn't we also tweak the comment on
RecoveryInProgress() as per the attached v2 diff?Yes, we should -- diff looks good to me, thanks.
Thanks for confirming, I've applied this to master.
--
Daniel Gustafsson https://vmware.com/
On Wed, Dec 1, 2021 at 8:24 AM Daniel Gustafsson <daniel@yesql.se> wrote:
The attached patch updates the code comment which is no longer true
after commit # 4a92a1c3d1c361ffb031ed05bf65b801241d7cddAgreed, but looking at this shouldn't we also tweak the comment on
RecoveryInProgress() as per the attached v2 diff?Yes, we should -- diff looks good to me, thanks.
Thanks for confirming, I've applied this to master.
Thanks both of you.
--
Robert Haas
EDB: http://www.enterprisedb.com