On Sun, 16 Aug 2026 at 09:13, Michael Paquier <[email protected]> wrote: > > On Fri, Aug 14, 2026 at 04:08:24PM +0200, Matthias van de Meent wrote: > > In line with various other efforts making things more re-entrant and > > thread-safe, here's a patch that makes stringToNode (and pg_strtok > > with it) thread safe. > > I have been reading the patch (not tested, just looking at the diffs), > and using a context as you are doing sounds like an acceptable thing > to do to remove the static pointer used by pg_strtok(). At least the > change makes sense here when taken in isolation.
Thanks for looking! > Any thoughts or comments from others? One update for the patch. This fixes non-DEBUG_NODE_TESTS_ENABLED builds; I forgot to update the READ_LOCATION_FIELD definition in those non-debug builds, CI picked this up quite quickly but I didn't get to fixing it until today. -Matthias
v2-0001-Make-stringToNode-infrastructure-thread-safe.patch
Description: Binary data
