On Fri, Dec 8, 2023 at 5:39 PM Alena Rybakina <lena.riback...@yandex.ru> wrote:
> On 06.12.2023 10:30, Richard Guo wrote: > > I've self-reviewed this patch again and I think it's now in a > > committable state. I'm wondering if we can mark it as 'Ready for > > Committer' now, or we need more review comments/feedbacks. > > > > To recap, this patch postpones reparameterization of paths until > > createplan.c, which would help avoid building the reparameterized > > expressions we might not use. More importantly, it makes it possible to > > modify the expressions in RTEs (e.g. tablesample) and in RelOptInfos > > (e.g. baserestrictinfo) for reparameterization. Failing to do that can > > lead to executor crashes, wrong results, or planning errors, as we have > > already seen. I marked it as 'Ready for Committer'. I think it is ready. Thank you. Appreciate that. Thanks Richard