On Sun, Sep 13, 2026 at 9:01 PM vignesh C <[email protected]> wrote:
>
>
> I preferred the single lookup approach. The
> v1-0002-Review-comment-fixes.patch at [1] has the changes for the
> same.

Thanks Vignesh.

Instead of isPublicationRelationExcept() with the exclusion logic
outside and the fallback logic inside -- which is also not very clear:

+ /* fallback to a non-exclusion entry for an undefined object */
+ return false;

I think we could introduce a function similar to
getProcedureTypeDescription() and getConstraintTypeDescription().
Please see the attached patch. Take the changes if you agree.

thanks
Shveta

Attachment: 0001-getPublicationRelationDescription-function.patch
Description: Binary data

Reply via email to