> On 14 Feb 2025, at 06:44, Michael Paquier wrote:
> the proposed patch makes sense to me.
Committed, with a backpatch down to 13.
--
Daniel Gustafsson
On Fri, Feb 14, 2025 at 12:15:40AM +0100, Daniel Gustafsson wrote:
> > On 13 Feb 2025, at 18:08, Dmitry Koval wrote:
> >
> > Hi!
> > Function EvalPlanQualFetchRowMark contains an assignment
> >
> > ExecRowMark *erm = earm->rowmark;
> >
> > before assert
> >
> > Assert(earm != NULL);
> >
> > M
> On 13 Feb 2025, at 18:08, Dmitry Koval wrote:
>
> Hi!
> Function EvalPlanQualFetchRowMark contains an assignment
>
> ExecRowMark *erm = earm->rowmark;
>
> before assert
>
> Assert(earm != NULL);
>
> Maybe these lines need to be swapped?
That does admittedly look a bit odd, that assertion c
