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
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 |