On Wed, Sep 22, 2021 at 5:03 PM [email protected] <[email protected]> wrote: > > Personally, I think we'd better move the code about changing publication's > tablelist into AlterPublicationTables and the code about changing > publication's > schemalist into AlterPublicationSchemas. It's similar to what the v29-patchset > did, the difference is the SET action, I suggest we drop all the tables in > function AlterPublicationTables when user only set schemas and drop all the > schema in AlterPublicationSchemas when user only set tables. In this approach, > we can keep schema and relation code separate and don't need to worry > about the locking order. > > Attach a top-up patch which refactor the code like above. >
Good suggestion. I think it would still be better if we can move the checks related to superuser and puballtables into a separate function that gets called before taking a lock on publication. -- With Regards, Amit Kapila.
