Re: Improve tab completion for REFRESH MATERIALIZED VIEW

From: Fujii Masao <masao(dot)fujii(at)gmail(dot)com>
To: Kevin Grittner <kgrittn(at)gmail(dot)com>
Cc: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Improve tab completion for REFRESH MATERIALIZED VIEW
Date: 2016-01-27 02:41:58
Message-ID: CAHGQGwFk2-6NWRnC2trokNhcnjkZH8pTmOG+JQ_HTApM6Mnc0Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jan 26, 2016 at 11:49 PM, Kevin Grittner <kgrittn(at)gmail(dot)com> wrote:
> On Tue, Jan 26, 2016 at 8:43 AM, Kevin Grittner <kgrittn(at)gmail(dot)com> wrote:
>
>> I will push something shortly with the
>> improvements from both of you, plus a couple other MV tab
>> completion issues I found in testing these patches.
>
> Done.

But ISTM that CREATE MATERIALIZED VIEW <tab> doesn't list
existing matviews yet. What's worse it lists existing *views*.

This happens because words_after_create mechanism doesn't support
the case where the keyword has more than one words like "MATERIALIZED VIEW".
Probably we should improve that mechanism so that even multiple words can be
handled. Or we should just add something like the following.

/* Complete CREATE MATERIALIZED VIEW with <name> */
+ else if (Matches3("CREATE", "MATERIALIZED", "VIEW"))
+ COMPLETE_WITH_SCHEMA_QUERY(Query_for_list_of_matviews, NULL);

Regards,

--
Fujii Masao

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2016-01-27 02:54:02 Re: Improve tab completion for REFRESH MATERIALIZED VIEW
Previous Message leo 2016-01-27 02:36:42 Does pglogical support PG 9.4.5?