Re: XID formatting and SLRU refactorings (was: Add 64-bit XIDs into PostgreSQL 15)

Alexander, Maxim,

Thank you for revisions.

On Thu, Nov 9, 2023 at 6:22 PM Maxim Orlov <orlovmg@gmail.com> wrote:
> Aleksander Alekseev,
>
> > Maxim,
> > I see both of us accounted for Alexanders feedback and submitted v59.
> > Your newer version seems to have issues on cfbot, so resubmitting the
> > previous patchset that passes the tests. Please feel free to add
> > changes.
>
> For unknown reasons, I do not receive any of your emails from after 2023-11-07 11:57:12 (Message-ID:
CAJ7c6TN1hKqNPGrNcq96SUyD=Z61raKGXF8iqq36qr90oudxRg@mail.gmail.com).
> Even after resend.
>
> Anyway, PFA patch set of version 61.  I've made some minor changes in the 0001 and add 004 in order to test actual
64-bitSLRU pages. 
>
> As for CF bot had failed on my v59 patch set, this is because of the bug.  It's manifested because of added 64-bit
pagestests. 
> The problem was in segno calculation, since we convert it from file name using strtol call.  But strtol return long,
> which is 4 byte long in x86.
>
> -           segno = (int) strtol(clde->d_name, NULL, 16);
> +           segno = strtoi64(clde->d_name, NULL, 16);

v61 looks good to me.  I'm going to push it as long as there are no objections.

------
Regards,
Alexander Korotkov