On Wed, Sep 29, 2021 at 9:07 AM houzj.f...@fujitsu.com
<houzj.f...@fujitsu.com> wrote:
>
> On Tues, Sep 28, 2021 10:46 PM vignesh C <vignes...@gmail.com> wrote:
> > Attached v34 patch has the changes for the same.
>
> 3)
> + /*
> + * Check if setting the relation to a different schema will result in
> the
> + * publication having schema and same schema's table in the
> publication.
> + */
> + if (stmt->objectType == OBJECT_TABLE)
> + {
> + ListCell *lc;
> + List *schemaPubids = GetSchemaPublications(nspOid);
> + foreach(lc, schemaPubids)
> + {
> + Oid pubid = lfirst_oid(lc);
> + if (list_member_oid(GetPublicationRelations(pubid,
> PUBLICATION_PART_ALL),
> + relid))
> + ereport(ERROR,
>
> How about we check this case like the following ?
>
> List *schemaPubids = GetSchemaPublications(nspOid);
> List *relPubids = GetRelationPublications(RelationGetRelid(rel));
> if (list_intersection(schemaPubids, relPubids))
> ereport(ERROR, ...
>
Won't this will allow changing one of the partitions for which only
partitioned table is part of the target schema? And then probably we
won't be able to provide the exact publication in the error message if
we followed the above?
--
With Regards,
Amit Kapila.
