On Wed, Jun 21, 2017 at 1:38 PM, Amit Khandekar <amitdkhan...@gmail.com> wrote: >>> Yep, it's more appropriate to use >>> ModifyTableState->rootResultRelationInfo->ri_RelationDesc somehow. That >>> is, if answer to the question I raised above is positive. > > From what I had checked earlier when coding that part, > rootResultRelInfo is NULL in case of inserts, unless something has > changed in later commits. That's the reason I decided to use the first > resultRelInfo.
We're just going around in circles here. Saying that you decided to use the first child's resultRelInfo because you didn't have a resultRelInfo for the parent is an explanation of why you wrote the code the way you did, but that doesn't make it correct. I want to know why you think it's correct. I think it's probably wrong, because it seems to me that if the INSERT code needs to use the parent's ResultRelInfo rather than the first child's ResultRelInfo, the UPDATE code probably needs to do the same. Commit d3cc37f1d801a6b5cad9bf179274a8d767f1ee50 got rid of resultRelInfos for non-leaf partitions, and commit e180c8aa8caf5c55a273d4a8e6092e77ff3cff10 added the resultRelInfo back for the topmost parent, because otherwise it didn't work correctly. If every partition in the hierarchy has a different attribute ordering, then it seems to me that it must surely matter which of those attribute orderings we pick. It's hard to imagine that we can pick *either* the parent's attribute ordering *or* that of the first child and nothing will be different - the attribute numbers inside the returning lists and WCOs we create have got to get used somehow, so surely it matters which attribute numbers we use, doesn't it? -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
