From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Arne Roland <A(dot)Roland(at)index(dot)de>, Amit Langote <amitlangote09(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Zhihong Yu <zyu(at)yugabyte(dot)com>, Julien Rouhaud <rjuju123(at)gmail(dot)com> |
Subject: | Re: missing indexes in indexlist with partitioned tables |
Date: | 2022-09-18 18:50:27 |
Message-ID: | CAApHDvqbH9XOUGz39S7W=Qn74t8-BYANd1aRvqR-VJFkj_frkA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, 18 Sept 2022 at 07:00, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> writes:
> > After a bit of trawling through the archives, I found it here:
> > https://www.postgresql.org/message-id/20180124162006.pmapfiznhgngwtjf%40alvherre.pgsql
> > I think there was insufficient discussion and you're probably right that
> > it wasn't the best fix. I don't object to finding another fix.
>
> FWIW, I don't see any big problem with what you did. We'd need to
> do something more like what David suggests if the planner ever has
> a reason to consider partitioned indexes. But as long as it does
> not, why expend the time to build data structures representing them?
Did you miss the report about left join removals not working with
partitioned tables due to lack of unique proofs? That seems like a
good enough reason to me.
> And we'd have to add code in quite a few places to ignore them,
> once they're in indexlist.
I think the same is true for "hypothetical" indexes. Maybe that would
be a good field to grep on to find the places that need to be
addressed.
David
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2022-09-18 20:57:45 | Re: Tree-walker callbacks vs -Wdeprecated-non-prototype |
Previous Message | Bharath Rupireddy | 2022-09-18 14:09:24 | Re: Refactor backup related code (was: Is it correct to say, "invalid data in file \"%s\"", BACKUP_LABEL_FILE in do_pg_backup_stop?) |