On Mon, Oct 19, 2020 at 8:48 PM Heikki Linnakangas <hlinn...@iki.fi> wrote: > On 19/10/2020 07:54, Amit Langote wrote: > > On Sun, Oct 18, 2020 at 12:54 AM Alvaro Herrera <alvhe...@alvh.no-ip.org> > > wrote: > >> Well, I was thinking on making the ri_PartitionInfo be about > >> partitioning in general, not just specifically for partition tuple > >> routing. Maybe Heikki is right that it may end up being simpler to > >> remove ri_PartitionInfo altogether. It'd just be a couple of additional > >> pointers in ResultRelInfo after all. > > > > So that's 2 votes for removing PartitionRoutingInfo from the tree. > > Okay, I have tried that in the attached 0002 patch. Also, I fixed > > some comments in 0001 that still referenced PartitionToRootMap. > > Pushed, with minor comment changes.
Thank you. > I also noticed that the way the getTargetResultRelInfo() helper function > was used, was a bit messy. It was used when firing AFTER STATEMENT > triggers, but for some reason the code to fire BEFORE STATEMENT triggers > didn't use it but duplicated the logic instead. I made that a bit > simpler, by always setting the rootResultRelInfo field in > ExecInitModifyTable(), making the getTargetResultRelInfo() function > unnecessary. Good, I was mildly annoyed by that function too. -- Amit Langote EDB: http://www.enterprisedb.com