Hi Michael,
On Tue, Nov 25, 2025 at 8:54 PM Michael Paquier <michael@paquier.xyz> wrote:
> > Tom, you are registered as a reviewer of the patch. The point of
> > contention of the patch, where I see there is no consensus yet, is if
> > my approach of using a redirection for the external TOAST pointers
> > with a new layer to facilitate the addition of more vartags (aka the
> > 64b value vartag proposed here, concept that could also apply to
> > compression methods later on) is acceptable. Moving to a different
> > approach, like the "brutal" one I am naming upthread where the
> > redirection layer is replaced by changes in all the code paths that
> > need to be touched, would be of course cheaper at runtime as there
> > would be no more redirection, but the maintenance would be a nightmare
> > the more vartags we add, and I have some plans for more of these.
> > Doing the switch would be a few hours work, so that would not be a big
> > deal, I guess. The important part is an agreement about the approach,
> > IMO.
>
> This point still got no reply. It would be nice to do something for
> this release regarding this old issue, IMO..
> --
Thanks for the updated v8. I’m also looking forward to feedback on the
vartag approach. My work on adding ZSTD compression depends on this
design decision, so consensus here will help me proceed in the right
direction.
--
Nikhil Veldanda
