On Fri, Sep 11, 2026 at 4:34 PM vignesh C <[email protected]> wrote: > > On Fri, 11 Sept 2026 at 11:21, Amit Kapila <[email protected]> wrote: > > > > On Thu, Sep 10, 2026 at 11:09 AM vignesh C <[email protected]> wrote: > > > > > > Finding #5: SET UNLOGGED on an excluded table produces an unrestorable > > > catalog state > > > > > > > + if (!toLogged && > > + GetRelationExcludedPublications(RelationGetRelid(rel)) != NIL) > > + ereport(ERROR, > > + (errcode(ERRCODE_OBJECT_NOT_IN_PREREQUISITE_STATE), > > + errmsg("cannot change table \"%s\" to unlogged because i > > > > Can we have one unified function to check whether a table is part of > > any publication and get an additional out parameter whether that is an > > include or exclude table? As we can't have the same table both > > included and excluded, this should work unless I am missing something. > > Here is an updated v4 version patch to handle the same. >
In my earlier example, there are both precept t and f. Won't your v4 new function return a random *isexcept result based on the first one it stumbles upon? IIUC, that `isexcept` out parameter should maybe be called `has_any_except`, and you should iterate until you find the first prexcept==true or find none. ====== [1] https://www.postgresql.org/message-id/CAHut%2BPunLReA86mPmi7Wr1-Hd_TnhtM1%2BKo8Bc6HhBz1LRjQNg%40mail.gmail.com Kind Regards, Peter Smith. Fujitsu Australia
