Re: [PATCH] psql: Add tab-complete for optional view parameters
Dean Rasheed <dean.a.rasheed@gmail.com>
From: Dean Rasheed <dean.a.rasheed@gmail.com>
To: Christoph Heiss <christoph@c8h4.io>
Cc: Jim Jones <jim.jones@uni-muenster.de>,
Mikhail Gribkov <youzhick@gmail.com>, PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2023-08-08T08:17:51Z
Lists: pgsql-hackers
On Mon, 7 Aug 2023 at 19:49, Christoph Heiss <christoph@c8h4.io> wrote: > > On Fri, Jan 06, 2023 at 12:18:44PM +0000, Dean Rasheed wrote: > > Hmm, I don't think we should be offering "check_option" as a tab > > completion for CREATE VIEW at all, since that would encourage users to > > use non-SQL-standard syntax, rather than CREATE VIEW ... WITH > > [CASCADED|LOCAL] CHECK OPTION. > > Left that part in for now. I would argue that it is a well-documented > combination and as such users would expect it to turn up in the > tab-complete as well. OTOH not against removing it either, if there are > others voicing the same opinion .. > On reflection, I think that's probably OK. I mean, I still don't like the fact that it's offering to complete with non-SQL-standard syntax, but that seems less bad than using an incomplete list of options, and I don't really have any other better ideas. Regards, Dean
Commits
-
psql: Add tab completion for view options.
- cd342474890f 17.0 landed
-
Fix typos in code and comments
- 5f6401f81cb2 16.0 cited