On Fri, Mar 01, 2024 at 01:16:37PM +0900, Kyotaro Horiguchi wrote: > This code intends to prevent a page header error from causing a record > reread, when a record is required to be read from multiple sources. We > could restrict this to only fire at segment boundaries. At segment > boundaries, we won't let LSN back up by using XLP_FIRST_IS_CONTRECORD. > > Having thought up to this point, I now believe that we should > completely prevent LSN from going back in any case. One drawback is > that the fix cannot be back-patched.
I don't immediately see why we should not be able to backpatch a fix that restricts the early page header check to happen only at segment boundary when we are in standby mode. Perhaps I'm just missing your point. If we cannot use that, wouldn't it be better just to stick with the proposed check on the TLI and force an error in the page read callback if the situation has changed due to a TLI jump with curFileTLI so as we don't loop and check infinitely these incorrect pages? -- Michael
signature.asc
Description: PGP signature
