On Fri, Jul 29, 2022 at 3:55 PM Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Fri, Jul 29, 2022 at 3:29 PM Alvaro Herrera <alvhe...@alvh.no-ip.org> > wrote: > > > > Well, I checked the commit and the functions I was talking about look OK > > now. However, looking again, pg_relation_is_publishable is in the wrong > > place (should be right below is_publishable_relaton), and I wonder why > > aren't get_publication_oid and get_publication_name in lsyscache.c. > > > > Right, both these suggestions make sense to me. Similarly, I think > functions get_subscription_name and get_subscription_oid should also > be moved to lsyscache.c. >
Attached, find a patch to address the above comments. Note that (a) I didn't change the comment atop pg_relation_is_publishable to refer to the actual function name instead of 'above' as it seems it can be an SQL variant for both the above functions. (b) didn't need to include pg_publication.h in lsyscache.c even after moving code to that file as the code is compiled even without that. -- With Regards, Amit Kapila.
v1-0001-Move-common-catalog-cache-access-routines-to-lsys.patch
Description: Binary data
