| From: | Richard Guo <guofenglinux(at)gmail(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Jaime Casanova <jcasanov(at)systemguards(dot)com(dot)ec>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Assert !bms_overlap(joinrel->relids, required_outer) |
| Date: | 2023-06-30 03:02:58 |
| Message-ID: | CAMbWs48A2GMGXjsjC-CMzpm62pztQr4aFS5VGbkoHViSCE35ow@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, Jun 30, 2023 at 12:16 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Pushed with that and defenses added to try_mergejoin_path and
> try_hashjoin_path. It looks like the various try_partial_xxx_path
> functions already reject cases that could be problematic. (They
> will not accept inner parameterization that would lead to the
> result being parameterized differently from the outer path.
> By induction, that should be fine.)
Thanks for pushing it!
Yeah, I also checked that and there is no problem with partial join
paths. However I found some opportunities for trivial revises there and
created a new patch for those at [1].
Thanks
Richard
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Kyotaro Horiguchi | 2023-06-30 03:05:17 | Re: vacuumdb/clusterdb/reindexdb: allow specifying objects to process in all databases |
| Previous Message | Amit Kapila | 2023-06-30 02:56:29 | Re: Assert while autovacuum was executing |