> > With this patch, the mapping is created *only once* during > RelationBuildPartitionDesc() to assign canonical indexes to individual > list values. The partition OID array will also be rearranged such that > using the new (canonical) index instead of the old > catalog-scan-order-based index will retrieve the correct partition for > that value. > > By the way, I fixed one thinko in your patch as follows: > > - result->oids[i] = oids[mapping[i]]; > + result->oids[mapping[i]] = oids[i];
While I can not spot any problem with this logic, when I make that change and run partition_join testcase in my patch, it fails because wrong partitions are matched for partition-wise join of list partitions. In that patch, RelOptInfo of partitions are saved in RelOptInfo of the parent by matching their OIDs. They are saved in the same order as corresponding OIDs. Partition-wise join simply joins the RelOptInfos at the same positions from both the parent RelOptInfos. I can not spot an error in this logic too. -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers