On Fri, Jul 17, 2026 at 9:50 AM Peter Smith <[email protected]> wrote: > > Hi Nisha. > > I noticed multiple places where these patches want to discover the > root partition table OID, but they are not interested in the ancestor > tree. > > I've created another thread/patch [1] to introduce a > `get_partition_root` C function. I think this could simplify some of > your code by removing the requirement to declare and free the unwanted > ancestor lists. > > Please take a look and give your +1 there if you think it would be useful. > > ====== > [1] > https://www.postgresql.org/message-id/CAHut%2BPtc73UpWSn4teMyMxQoHwGJjRwhWiu%3DhGJqAj5k2AOXEw%40mail.gmail.com >
Nisha, Peter, Do you think we should make get_partition_root's patch as 0001 in this thread and rebase rest of patches atop it? That way duplicated code in many places will be removed. Another idea is to make get_partition_root's patch as last patch here, but I prefer it as first patch. Thoughts? thanks Shveta
