On Tue, Jan 26, 2016 at 11:49 PM, Kevin Grittner <kgri...@gmail.com> wrote: > On Tue, Jan 26, 2016 at 8:43 AM, Kevin Grittner <kgri...@gmail.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 -- Sent via pgsql-hackers mailing list (email@example.com) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers