Hi,

On Fri, 3 Jul 2026 at 00:21, Nathan Bossart <[email protected]>
wrote:

> This infrastructure was added by commit 722408bcd1 and was only ever used
> for publications and subscriptions.  Since we just bumped psql's minimum
> supported version to v10, it's no longer used by anything.  Furthermore, a
> quick glance indicates there's nothing else that should use it.  So, IMHO
> we might as well remove it.
>

Thanks for the patch!

Doing a grep on VersionedQuery I could find one instance missing
in your patch in 010_tab_completion.pl file there's a test block

# check VersionedQuery infrastructure
check_completion(
    "DROP PUBLIC\t \t\t",
    qr/DROP PUBLICATION\s+some_publication /,
    "check VersionedQuery");

I think the test case is fine, but the comment + label needs to be
updated or case drop if it's redundant.

Rest, LGTM.

Regards,
Ayush

Reply via email to