Hi,
On 2017-06-19 00:30:26 -0700, Andres Freund wrote:
> There seems to be a larger question ehre though: Why does
> XLogFileReadAnyTLI() probe all timelines even if they weren't a parent
> at that period? That seems like a bad idea, especially in more
> complicated scenarios where some precursor timeline might live for
> longer than it was a parent? ISTM XLogFileReadAnyTLI() should check
> which timeline a segment ought to come from, based on the historY?
One thing that I blamed first, before debunking it, is that after
promotion we do:
/* * Preallocate additional log files, if wanted. */PreallocXlogFiles(EndOfLog);
where EndOfLog points to the last replayed record, rather than last
record(s). I think that's currently harmless, but it's certainly
fragile. Given the uselessness of PreallocXlogFiles() calls, I'm
inclined to just remove it here...
- Andres
