Re: MemoizePath fails to work for partitionwise join

2022-12-05 Thread Richard Guo
On Tue, Dec 6, 2022 at 1:42 AM Tom Lane wrote: > One small problem is that top_parent doesn't exist in the back branches, > so I had to substitute a much uglier lookup in order to make this work > there. I'm surprised that we got away without top_parent for this long > TBH, but anyway this fix v

Re: MemoizePath fails to work for partitionwise join

2022-12-05 Thread Tom Lane
Richard Guo writes: > The fix is straightforward, just to use outerrel->top_parent if it is > not null and leave the reparameterization work to try_nestloop_path. In > addition, I believe when reparameterizing MemoizePath we need to adjust > its param_exprs too. Right you are. I'd noticed the a