Re: Miscellaneous tab completion issue fixes

From: Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>
To: vignesh C <vignesh21(at)gmail(dot)com>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Miscellaneous tab completion issue fixes
Date: 2022-10-04 09:35:25
Message-ID: 8735c4kk2q.fsf@wibble.ilmari.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

vignesh C <vignesh21(at)gmail(dot)com> writes:

> On Tue, 4 Oct 2022 at 09:13, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>>
>> On Mon, Oct 03, 2022 at 06:29:32PM +0100, Dagfinn Ilmari Mannsåker wrote:
>> > vignesh C <vignesh21(at)gmail(dot)com> writes:
>> >> + else if (TailMatchesCS("\\dRp*"))
>> >> + COMPLETE_WITH_QUERY(Query_for_list_of_publications[0].query);
>> >> + else if (TailMatchesCS("\\dRs*"))
>> >> + COMPLETE_WITH_QUERY(Query_for_list_of_subscriptions[0].query);
>> >
>> > These are version-specific queries, so should be passed in their
>> > entirety to COMPLETE_WITH_VERSIONED_QUERY() so that psql can pick the
>> > right version, and avoid sending the query at all if the server is too
>> > old.
>>
>> +1.
>
> Modified
>
> >> +/* add these to Query_for_list_of_roles in OWNER TO contexts */
>> >> +#define Keywords_for_list_of_owner_to_roles \
>> >> +"CURRENT_ROLE", "CURRENT_USER", "SESSION_USER"
>> >
>> > I think this would read better without the TO, both in the comment and
>> > the constant name, similar to the below only having GRANT without TO:
>>
>> Keywords_for_list_of_grant_roles is used in six code paths, so it
>> seems to me that there is little gain in having a separate #define
>> here. Let's just specify the list of allowed roles (RoleSpec) where
>> OWNER TO is parsed.
>
> I have removed the macro and specified the allowed roles.
>
> Thanks for the comments, the attached v2 patch has the changes for the same.

LGTM, +1 to commit.

- ilmari

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Nikita Malakhov 2022-10-04 10:45:38 Re: Pluggable toaster
Previous Message Laurenz Albe 2022-10-04 09:33:15 Re: future of serial and identity columns