> On 19 Sep 2023, at 11:21, bt23nguyent <[email protected]> wrote:
> Please let me know if you have any further suggestions that I can improve
> more.
+ *logdetail = pstrdup("WAL archiving failed because
basic_archive.archive_directory is not set");
Nitpick: detail messages should end with a period per the error message style
guide [0].
- archiving will proceed only when it returns <literal>true</literal>.
+ archiving will proceed only when it returns <literal>true</literal>. The
+ archiver may also emit the detail explaining how the module is not
configured
+ to the sever log if the archive module has any.
I think this paragraph needs to be updated to include how the returned
logdetail is emitted, since it currently shows the WARNING without mentioning
the added detail in case returned. It would also be good to mention that it
should be an allocated string which the caller can free.
--
Daniel Gustafsson
[0] https://www.postgresql.org/docs/devel/error-style-guide.html
