Re: [HACKERS] Tab completion support for ALTER SUBSCRIPTION REFRESH PUBLICATION - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: [HACKERS] Tab completion support for ALTER SUBSCRIPTION REFRESH PUBLICATION
Date
Msg-id CAD21AoD7sfAP58e-eYOqv4_D9gAQ2C6E7qqSVsJeE8GRVmDMHg@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] Tab completion support for ALTER SUBSCRIPTION REFRESH PUBLICATION  (Fujii Masao <masao.fujii@gmail.com>)
Responses Re: [HACKERS] Tab completion support for ALTER SUBSCRIPTION REFRESH PUBLICATION  (Fujii Masao <masao.fujii@gmail.com>)
List pgsql-hackers
On Wed, Apr 12, 2017 at 11:48 PM, Fujii Masao <masao.fujii@gmail.com> wrote:
> On Wed, Apr 12, 2017 at 5:12 PM, Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>> Hi,
>>
>> I've attached a patch for $subject. Please check it.
>
> + COMPLETE_WITH_LIST8("WITH", "CONNECTION", "SET PUBLICATION", "ENABLE",
> + "DISABLE", "OWNER TO", "RENAME TO", "REFRESH PUBLICATION WITH");
>
> "WITH" should be "WITH ("?
> Also "REFRESH PUBLICATION WITH" should be "REFRESH PUBLICATION WITH ("?

Agreed. I found other bugs of tab completion of PUB/SUB DDLs. Attached
latest patch incorporated these fixes.

>
> BTW, I found that ALTER SUBSCRIPTION ... RENAME TO exists in tab-complete.c,
> but not in the documentation. ALTER PUBLICATION has the same issue, too.
> So I think that we need to apply the attached patch which improves the
> documentation
> for ALTER PUBLICATION and SUBSCRIPTION.

+1

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center

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

Attachment

pgsql-hackers by date:

Previous
From: Jeff Davis
Date:
Subject: Re: [HACKERS] Merge join for GiST
Next
From: David Rowley
Date:
Subject: Re: [HACKERS] Foreign Join pushdowns not working properly for outer joins