Re: Feature improvement of tab completion for DEALLOCATE

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: btnakamichin <btnakamichin(at)oss(dot)nttdata(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: Feature improvement of tab completion for DEALLOCATE
Date: 2020-09-28 02:31:42
Message-ID: d12635e7-69b9-15fd-3b6d-014645ffb851@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2020/09/25 15:18, btnakamichin wrote:
> 2020-09-25 14:30 に Fujii Masao さんは書きました:
>> On 2020/09/25 13:45, btnakamichin wrote:
>>> Hello!
>>>
>>> I’d like to improve the deallocate tab completion.
>>>
>>> The deallocate tab completion can’t complement “ALL”. Therefore, this patch fixes the problem.
>>
>> Thanks for the patch! It looks basically good, but I think it's better
>> to add newline just before UNION to avoid long line, as follows.
>>
>> - COMPLETE_WITH_QUERY(Query_for_list_of_prepared_statements);
>> + COMPLETE_WITH_QUERY(Query_for_list_of_prepared_statements
>> +                                                       " UNION SELECT 'ALL'");
>>
>> Regards,
>
> Thank you, I appreciate your comment.
>
> I have attached pattch with newline.

Thanks for updating the patch! But it contains only the diff from the previous patch.
Anyway, I pushed the change that you proposed. Thanks!

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2020-09-28 02:35:06 Re: Small improvements to pg_list.h's linitial(), lsecond(), lthird() etc macros
Previous Message Keisuke Kuroda 2020-09-28 02:20:42 Re: Logical replication CPU-bound with TRUNCATE/DROP/CREATE many tables