From: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
---|---|
To: | "tanghy(dot)fnst(at)fujitsu(dot)com" <tanghy(dot)fnst(at)fujitsu(dot)com> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Remove "FROM" in "DELETE FROM" when using tab-completion |
Date: | 2021-05-11 08:44:02 |
Message-ID: | CAFiTN-ubS-TNwUpeAk8AeaYqgaT5HJ+gArfLA-UqsOqPCsqjww@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, May 11, 2021 at 1:00 PM tanghy(dot)fnst(at)fujitsu(dot)com
<tanghy(dot)fnst(at)fujitsu(dot)com> wrote:
>
> On Tuesday, May 11, 2021 2:53 PM, Michael Paquier <michael(at)paquier(dot)xyz> wrote
> > else if (TailMatches("DELETE", "FROM", MatchAny))
> > COMPLETE_WITH("USING", "WHERE");
> >- /* XXX: implement tab completion for DELETE ... USING */
> >
> >Why are you removing that? This sentence is still true, no?
>
> IIRC, XXX in comment is used to flag something that is bogus but works.
> When the sentence introduced here in f5ab0a14, the fix for "DELETE ... USING" is not as good as it is now.(I guess that's why the comment was added). And for now, IMHO, we can remove the comment directly.
But your patch is doing nothing to add the implementation for DELETE..
USING. Basically, the tab completion support for DELETE....USING is
still pending right?
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Amul Sul | 2021-05-11 08:45:32 | Re: [Patch] ALTER SYSTEM READ ONLY |
Previous Message | Fujii Masao | 2021-05-11 08:41:53 | Re: compute_query_id and pg_stat_statements |