On Wed, Dec 31, 2025 at 04:02:24PM +0100, Dharin Shah wrote: > Let me know what you’d prefer for next steps: keep this patch as a > detoast-local refactor, or respin it to align more directly with a shared > decoded external-pointer interface in the direction of the 8‑byte work.
My apologies for the rather long silence on this thread. As the next step of this project, I am going to put my hands of what you are suggesting here, and see how I can align it with the 64-bit toast value patch: https://www.postgresql.org/message-id/caoj6k6devi0nvljmldhyrjs_n_nzo5d_ou89ao1u53u6ncd...@mail.gmail.com What I am pretty sure about at this stage is that there is little love for the patch set I have sent on the other thread where I have been using pointer redirections for the TOAST function calls with callbacks (perhaps I'll be able to apply some of the renaming patches anyway, nobody would scream at me for that), at least nobody has put a +1 on it or just ignored it, so this approach feels dead to me. What you are suggesting upthread, though, is a direction I'd like to dig into and this comes down to how I can unify what you want to do for zstd and what I want to do with Oid8. Perhaps that you are right and that it is just simpler to invest on an interface in the detoast code, but I still see that there is nothing done for the logical decoding or amcheck code paths, which is something my other patch is able to deal with transparently. -- Michael
signature.asc
Description: PGP signature
