On Thu, Oct 1, 2026 at 10:26 AM Zsolt Parragi <zsolt.parragi@percona.com> 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