The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: not tested
Documentation: not tested
I've looked at this patch and don't see any problems with it. It is minimally invasive, it doesn't affect functionality
unlessanyone (e.g. extension) sets its own slotOps in CustomScanState.
Furthermore, the current patch very slightly modifies patch 0b03e5951bf0 with the intention of introducing
extensibility.So I think adding more extensibility regarding different tuple formats is an excellent thing to do.
I'm going to mark it as RfC if there are no objections.
Kind regards,
Pavel Borisov,
Supabase
The new status of this patch is: Ready for Committer