On Fri, 14 Oct 2022 at 15:15, Richard Guo <guofenglinux@gmail.com> wrote:
> The v3 patch looks good to me.
Thank you for looking at that.
One other thought I had about the duplicate "Limit" node in the final
plan was that we could make the limit clause an Expr like
LEAST(<existing limit clause>, 1). That way we could ensure we get at
most 1 row, but perhaps less if the expression given in the LIMIT
clause evaluated to 0. This will still work correctly when the
existing limit evaluates to NULL. I'm still just not that keen on this
idea as it means still having to either edit the parse's limitCount or
store the limit details in a new field in PlannerInfo and use that
when making the final LimitPath. However, I'm still not sure doing
this is worth the extra complexity.
If nobody else has any thoughts on this or the patch in general, then
I plan to push it in the next few days.
David