Re: [PATCH] Tiny CREATE STATISTICS tab-completion cleanup

Поиск
Список
Период
Сортировка
От Tomas Vondra
Тема Re: [PATCH] Tiny CREATE STATISTICS tab-completion cleanup
Дата
Msg-id 65551c69-46d4-305e-25fb-2e093454d660@2ndquadrant.com
обсуждение исходный текст
Ответ на [PATCH] Tiny CREATE STATISTICS tab-completion cleanup  (ilmari@ilmari.org (Dagfinn Ilmari Mannsåker))
Ответы Re: [PATCH] Tiny CREATE STATISTICS tab-completion cleanup  (Tomas Vondra <tomas.vondra@2ndquadrant.com>)
Список pgsql-hackers
Hi,

On 11/26/18 5:49 PM, Dagfinn Ilmari Mannsåker wrote:
> Hi Hackers,
> 
> As I was hacking on the CREATE TABLE tab completions, I noticed that the
> CREATE STATISTICS completion was checking manually for the start and end
> of the parenthesised list instead of using the "(*)" wildcard (because
> it predates that functionality).  Attached is a patch that updates it to
> use the modern syntax.
> 

Makes sense. At first I was wondering why it was not modified by the
patch introducing the "(*)" wildcard, but I see 121213d9 only cared
about VACUUM, EXPLAIN and ANALYZE.

The patch seems fine to me, I'll get it committed.


regards

-- 
Tomas Vondra                  http://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


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

Предыдущее
От: Andres Freund
Дата:
Сообщение: Re: IMMUTABLE and PARALLEL SAFE function markings
Следующее
От: Joe Conway
Дата:
Сообщение: Re: pg_config wrongly marked as not parallel safe?