Hi Richard, On Tue, Jun 6, 2023 at 12:20 PM Richard Guo <guofengli...@gmail.com> wrote: > On Mon, Jun 5, 2023 at 9:19 PM Etsuro Fujita <etsuro.fuj...@gmail.com> wrote: >> To avoid this issue, I am wondering if we should modify >> add_paths_to_joinrel() in back branches so that it just disallows the >> FDW to consider pushing down joins when the restrictlist has >> pseudoconstant clauses. Attached is a patch for that.
> I think we can do that in back branches. But I'm a little concerned > that we'd miss a better plan if FDW cannot push down joins in such > cases. I may be worrying over nothing though if it's not common that > the restrictlist has pseudoconstant clauses. Yeah, it is unfortunate that we would not get better plans. Given that it took quite a long time to find this issue, I suppose that users seldom do foreign joins with pseudoconstant clauses, though. Anyway thanks for working on this, Richard! Best regards, Etsuro Fujita
