On Mon, May 14, 2018 at 12:57 AM, David Rowley <david.row...@2ndquadrant.com> wrote: > I noticed that a comment in get_partition_dispatch_recurse claims that: > > "it contains the > * leaf partition's position in the global list *leaf_part_oids minus 1" > > The "minus 1" part is incorrect. It simply just stores the 0-based > index of the item in the list. I was going to fix it by removing just > that part, but instead, I ended up rewriting the whole thing.
I think that's clearer. Committed with a few tweaks that are hopefully improvements. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company