On Thu, 29 Oct 2026 at 21:42, Greg Burd <[email protected]> wrote:
>
> Open item from review:
>
> Amit suggested [4] registering bucket_blkno only for the overflow-page
> cases, since for a primary-bucket-page record the block is already
> known, saving 4 bytes on the common records.  I have not made that
> change yet; I can in the next version if we agree it is worth the extra
> branch in each redo path.

Yes, that would be better. The
v3-0002-Append-the-hash-primary-bucket-block-number-to-WA.patch patch
has the changes for the same. The v3-0001 is the same as v2-0001
posted by Greg.  Please merge it if you feel the changes are ok.
I did not include the v2-0002-Remove-REGBUF_NO_CHANGE patch as we can
get to it once we bring the first patch into committable shape.

Regards,
Vignesh

Attachment: v3-0002-Append-the-hash-primary-bucket-block-number-to-WA.patch
Description: Binary data

Attachment: v3-0001-Don-t-register-pin-only-primary-bucket-pages-in-h.patch
Description: Binary data

Reply via email to