Hi,

SharedRecoveryState member of XLogCtl is no longer a boolean flag, got changes
in 4e87c4836ab9 to enum but, comment referring to it still referred as the
boolean flag which is pretty confusing and incorrect.

Also, the last part of the same comment is as:

" .. although the boolean flag to allow WAL is probably atomic in
itself, .....",

I am a bit confused here too about saying "atomic" to it, is that correct?
I haven't done anything about it, only replaced the "boolean flag" to "recovery
state" in the attached patch.

Regards,
Amul

Attachment: fix_comment_in_StartupXLOG.patch
Description: Binary data

Reply via email to