On Fri, Oct 25, 2019 at 02:07:04AM +0000, tsunakawa.takay@fujitsu.com wrote:
> From: Simon Riggs <simon@2ndquadrant.com>
>> From xlogdefs.h added by 9155580:
>> /*
>> * First LSN to use for "fake" LSNs.
>> *
>> * Values smaller than this can be used for special per-AM purposes.
>> */
>> #define FirstNormalUnloggedLSN ((XLogRecPtr) 1000)
>
> Yeah, I had seen it, but I didn't understand what kind of usage is assumed.
There is an explanation in the commit message of 9155580: that's to
make an interlocking logic in GiST able to work where a valid LSN
needs to be used. So a magic value was just wanted.
Your patch looks fine to me by the way after a second look, so I think
that we had better commit it and back-patch sooner than later. If
there are any objections or more comments, please feel free..
--
Michael