Hi,

On 2026-08-16 16:13:20 +0900, Michael Paquier wrote:
> On Fri, Aug 14, 2026 at 04:08:24PM +0200, Matthias van de Meent wrote:
> > There are other things we could do to improve this code (e.g.
> > de/serializaing based on node descriptors instead of generated
> > functions), but I consider such improvements to be out of scope for
> > this patch; the attached is just a simple move from global variables
> > to a context struct passed down the call stack.
> 
> That sounds fair.
> 
> Any thoughts or comments from others?

Tackling those things separately makes a lot of sense to me. Not using static
variables for the node infrastructure also makes a lof of sense, I find APIs
like pg_strtok() quite terrible, even disregarding thread safety.

Greetings,

Andres Freund


Reply via email to