Re: Remove "FROM" in "DELETE FROM" when using tab-completion

Поиск
Список
Период
Сортировка
От Julien Rouhaud
Тема Re: Remove "FROM" in "DELETE FROM" when using tab-completion
Дата
Msg-id 20210510054815.hlqjai7mc6sum7ay@nol
обсуждение исходный текст
Ответ на Remove "FROM" in "DELETE FROM" when using tab-completion  ("tanghy.fnst@fujitsu.com" <tanghy.fnst@fujitsu.com>)
Ответы Re: Remove "FROM" in "DELETE FROM" when using tab-completion  (Dilip Kumar <dilipbalaut@gmail.com>)
RE: Remove "FROM" in "DELETE FROM" when using tab-completion  ("tanghy.fnst@fujitsu.com" <tanghy.fnst@fujitsu.com>)
Список pgsql-hackers
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.



В списке pgsql-hackers по дате отправления:

Предыдущее
От: "tanghy.fnst@fujitsu.com"
Дата:
Сообщение: Remove "FROM" in "DELETE FROM" when using tab-completion
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: Parallel INSERT SELECT take 2