On Thu, Aug 6, 2020 at 2:30 AM Tom Lane <t...@sss.pgh.pa.us> wrote: > Amit Langote <amitlangot...@gmail.com> writes: > > The attached patch should fix that. > > I don't like this patch at all though; I do not think it is being nearly > careful enough to ensure that it's matched the surviving relation OIDs > correctly. In particular it blithely assumes that a zero in relid_map > *must* match the immediately next entry in partdesc->oids, which is easy > to break if the new partition is adjacent to the one the planner managed > to prune.
Indeed, you're right. > So I think we should do it more like the attached. Thanks for pushing that. -- Amit Langote EnterpriseDB: http://www.enterprisedb.com
