On Thu, Jan 16, 2020 at 11:17:36PM +0900, Fujii Masao wrote: > OK, I updated the patch that way. > Attached is the updated version of the patch.
Thanks. I have few tweaks to propose to the docs. + raise a PANIC-level error, aborting the recovery. Setting Instead of "PANIC-level error", I would just use "PANIC error", and instead of "aborting the recovery" just "crashing the server". + causes the system to ignore those WAL records WAL records are not ignored, but errors caused by incorrect page references in those WAL records are. The current phrasing sounds like the WAL records are not applied. Another thing that I just recalled. Do you think that it would be better to mention that invalid page references can only be seen after reaching the consistent point during recovery? The information given looks enough, but I was just wondering if that's worth documenting or not. -- Michael
signature.asc
Description: PGP signature