Re: psql tabcomplete - minor bugfix - tabcomplete for SET ROLE TO xxx

Поиск
Список
Период
Сортировка
От Pavel Stehule
Тема Re: psql tabcomplete - minor bugfix - tabcomplete for SET ROLE TO xxx
Дата
Msg-id CAFj8pRDrrjsKQnnogjPKZttqpO-89Ue28iR47P+n9iwch1kkCg@mail.gmail.com
обсуждение исходный текст
Ответ на Re: psql tabcomplete - minor bugfix - tabcomplete for SET ROLE TO xxx  (Jeevan Chalke <jeevan.chalke@gmail.com>)
Ответы Re: psql tabcomplete - minor bugfix - tabcomplete for SET ROLE TO xxx
Список pgsql-hackers
Дерево обсуждения


2015-05-29 9:28 GMT+02:00 Jeevan Chalke <jeevan.chalke@gmail.com>:
The following review has been posted through the commitfest application:
make installcheck-world:  not tested
Implements feature:       tested, passed
Spec compliant:           tested, passed
Documentation:            not tested

I agree with Peter that "We don't tab-complete everything we possibly could", but using tabs after "SET ROLE TO " provides "DEFAULT" as an option which seems wrong.
This patch adds list of roles over there, which I guess good to have than giving something unusual.

Surely, we cannot to support tab-complete everywhere. But if we can do it simply, we should to do it. Why:

1. It is good help for beginners

2. I am PostgreSQL lecture and evangelist in Czech Republic and Slovak Republic. The tabcomplete possibility is surprisingly good factor for accepting PostgreSQL concept, architecture, psql 

But back to this topic. I am thinking so it is little bit different due fact so we support two very syntax for one feature. And looks little bit strange, so one way is supported by autocomplete and second not.
 

I reviewed this straight forward patch and looks good to me.

Since we might not want this, review is done and thus passing it to committer to decide. 

ok
 

The new status of this patch is: Ready for Committer

Thank you very much

Regards

Pavel


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

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Jeevan Chalke
Дата:
Сообщение: Re: psql tabcomplete - minor bugfix - tabcomplete for SET ROLE TO xxx
Следующее
От: Michael Paquier
Дата:
Сообщение: Re: RFC: Remove contrib entirely