Re: [PATCH] psql: Add tab-complete for optional view parameters

From: "Gregory Stark (as CFM)" <stark(dot)cfm(at)gmail(dot)com>
To: Mikhail Gribkov <youzhick(at)gmail(dot)com>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, Christoph Heiss <christoph(at)c8h4(dot)io>
Subject: Re: [PATCH] psql: Add tab-complete for optional view parameters
Date: 2023-03-20 18:40:43
Message-ID: CAM-w4HNoGuCWukQysXa8GUekHrzXDpTHVZLkED+s_4RaGmQbfQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, 29 Jan 2023 at 05:20, Mikhail Gribkov <youzhick(at)gmail(dot)com> wrote:
>
> The problem is obviously in the newly added second line of the following clause:
> COMPLETE_WITH("ALTER COLUMN", "OWNER TO", "RENAME",
> "SET SCHEMA", "SET (", "RESET (");
>
> "set schema" and "set (" alternatives are competing, while completion of the common part "set<space>" leads to a string composition which does not have the check branch (Matches("ALTER", "VIEW", MatchAny, "SET")).
>
> I think it may worth looking at "alter materialized view" completion tree and making "alter view" the same way.
>
> The new status of this patch is: Waiting on Author

I think this patch received real feedback and it looks like it's clear
where there's more work needed. I'll move this to the next commitfest.
If you plan to work on it this month we can always move it back.

--
Gregory Stark
As Commitfest Manager

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeff Davis 2023-03-20 18:41:15 Re: Request for comment on setting binary format output per session
Previous Message Gregory Stark (as CFM) 2023-03-20 18:36:33 Re: Make ON_ERROR_STOP stop on shell script failure