Re: (LOCK TABLE options) “ONLY” and “NOWAIT” are not yet implemented

From: Shinya Kato <katousnk(at)oss(dot)nttdata(dot)com>
To: bt21tanigaway <bt21tanigaway(at)oss(dot)nttdata(dot)com>
Cc: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Shinya11(dot)Kato(at)nttdata(dot)com
Subject: Re: (LOCK TABLE options) “ONLY” and “NOWAIT” are not yet implemented
Date: 2021-09-28 13:46:35
Message-ID: c9cb8e6b4ff37e1bd7e4746964eef05b@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2021-09-28 17:06 に bt21tanigaway さんは書きました:
> 2021-09-28 17:03 に bt21tanigaway さんは書きました:
>> 2021-09-28 16:36 に Fujii Masao さんは書きました:
>>> On 2021/09/28 16:13, bt21tanigaway wrote:
>>>> Hi,
>>>>
>>>> (LOCK TABLE options) “ONLY” and “NOWAIT” are not yet implemented in
>>>> tab-complete. I made a patch for these options.
>>>
>>> Thanks for the patch!
>>> The patch seems to forget to handle the tab-completion for
>>> "LOCK ONLY <table-name> IN".
>>
>> Thanks for your comment!
>> I attach a new patch fixed to this mail.
>>
>> Regards,
>>
>> Koyu Tanigawa
>
> Sorry, I forgot to attach patch file.
> "fix-tab-complete2.patch" is fixed!
>
> Regards,
>
> Koyu Tanigawa
Thank you for your patch.
I have two comments.

1. When I executed git apply, an error occured.
---
$ git apply ~/Downloads/fix-tab-complete2.patch
/home/penguin/Downloads/fix-tab-complete2.patch:14: indent with spaces.
COMPLETE_WITH_SCHEMA_QUERY(Query_for_list_of_tables, " UNION SELECT
'TABLE'" " UNION SELECT 'ONLY'");
warning: 1 line adds whitespace errors.
---

2. The command "LOCK TABLE a, b;" can be executed, but tab-completion
doesn't work properly. Is it OK?

--
Regards,

--
Shinya Kato
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2021-09-28 13:48:50 Re: Trap errors from streaming child in pg_basebackup to exit early
Previous Message Magnus Hagander 2021-09-28 13:38:50 Re: [PATCH] Support pg_ident mapping for LDAP