On Thu, Oct 1, 2026 at 10:26 AM Zsolt Parragi <[email protected]> wrote: > But regardless to this, CreateExecutorState documents on all branches > that callers are expected to set es_snapshot: > > /* > * Initialize all fields of the Executor State structure > */ > estate->es_direction = ForwardScanDirection; > estate->es_snapshot = InvalidSnapshot; /* caller must initialize > this */ > estate->es_crosscheck_snapshot = InvalidSnapshot; /* no > crosscheck */ > > So I think it would be good practice to set it, independently to this bug.
Agreed, thanks for finding this; the PR for this issue is in flight. David
