tab completion for alter extension

Started by Jeff Janesabout 10 years ago3 messageshackers
Jump to latest
#1Jeff Janes
jeff.janes@gmail.com

tab completion for "alter extension foobar update" yields a list of
tables. That is actively misleading. (This is not new in 9.6.)

It should complete to nothing, or "TO" followed by a list of available versions.

The attached patch takes approach 2. I thought of adding a ";" to the
list completions to signify you can optionally end the command
immediately after UPDATE and have it be a complete command. But I
thought perhaps that was too clever, and unprecedented.

Will add to commitfest-next

Cheers,

Jeff

Attachments:

alter_extension_update_tab.ctext/x-csrc; charset=US-ASCII; name=alter_extension_update_tab.cDownload+21-0
In reply to: Jeff Janes (#1)
Re: tab completion for alter extension

The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation: tested, passed

I did some tests and found nothing special. The stated resource is implemented correctly.
He passes all regression tests and enables the use of the new features specified.

The new status of this patch is: Ready for Committer

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Gerdan Rezende dos Santos (#2)
Re: tab completion for alter extension

Gerdan Santos <gerdan@gmail.com> writes:

I did some tests and found nothing special. The stated resource is implemented correctly.
He passes all regression tests and enables the use of the new features specified.

The new status of this patch is: Ready for Committer

Pushed, thanks.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers