> > > Any opinions on this? > > > > I will take a closer look at this in a bit. > > Patch looks good to me. One nit: > > + table_oid = IndexGetRelation(child_oid, true); > + if (!OidIsValid(table_oid)) > + continue; > > How about a short comment on why this is needed even with a lock on > the parent, since that lock doesn't cover lower-level leaves in a > multi-level partition tree?
I think it's worth a comment, too. Patch LGTM. -- Regards, ChangAo Chen
