Re: Remove "FROM" in "DELETE FROM" when using tab-completion
От | Julien Rouhaud |
---|---|
Тема | Re: Remove "FROM" in "DELETE FROM" when using tab-completion |
Дата | |
Msg-id | 20210510055906.yeaasc7n7kt3qmjx@nol обсуждение исходный текст |
Ответ на | Re: Remove "FROM" in "DELETE FROM" when using tab-completion (Dilip Kumar <dilipbalaut@gmail.com>) |
Список | pgsql-hackers |
On Mon, May 10, 2021 at 11:21:11AM +0530, Dilip Kumar wrote: > On Mon, May 10, 2021 at 11:17 AM Julien Rouhaud <rjuju123@gmail.com> wrote: > > > > On Mon, May 10, 2021 at 05:36:35AM +0000, tanghy.fnst@fujitsu.com wrote: > > > You see, the tab-completion for "DELETE" is "DELETE FROM" which is not same as help-command said(which is "DELETE"). > > > I tried to figure out why "FROM" is introduced here, but no good result got. In [1] someone changed "DELETE" to "DELETEFROM" but no reason added. > > > > > > IMO, the "FROM" is unnecessary just like "INTO" for "INSERT" command. So I tried to fix the inconsistency by removing"FROM" from "DELETE FROM" in tab-complete.c. > > > Please see the attached patch. Any comment or different thought is very welcome. > > > > I think the behavior now is correct. The goal of autocompletion is to save > > keystrokes and time. As the only valid keyword after a DELETE (at least in a > > DeleteStmt) is FROM, it's a good thing that you get back "DELETE FROM" directly > > rather than asking that to autocomplete in multiple steps. > > > > Now, the \help command is for commands, which is a different thing as the > > command in that case is DELETE not DELETE FROM, even if you will have to follow > > your DELETE with a FROM. > > I agree with Julien. But, I also agree with the consistency point > from Tang. So maybe we can fix the insert and add INSERT INTO in the > tab completion? +1 for that.
В списке pgsql-hackers по дате отправления: