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

From: bt21tanigaway <bt21tanigaway(at)oss(dot)nttdata(dot)com>
To: Shinya11(dot)Kato(at)nttdata(dot)com
Cc: masao(dot)fujii(at)oss(dot)nttdata(dot)com, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: (LOCK TABLE options) “ONLY” and “NOWAIT” are not yet implemented
Date: 2021-10-04 02:17:19
Message-ID: d4e33e4f465d67c6e723a7f45eeb1d43@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

>> else if (Matches("LOCK", MatchAny, "IN", "ACCESS|ROW") ||
>> - Matches("LOCK", "TABLE", MatchAny, "IN", "ACCESS|ROW"))
>> + Matches("LOCK", "TABLE", MatchAny, "IN", "ACCESS|ROW") ||
>> + Matches("LOCK", "ONLY", MatchAny, "IN", "ACCESS|ROW") ||
>> + Matches("LOCK", "TABLE", "ONLY", MatchAny, "IN", "ACCESS|ROW"))
> I think this code is redundant, so I change following.
> ---
> else if (HeadMatches("LOCK") && TailMatches("IN", "ACCESS|ROW"))
> ---
> I created the patch, and attached it. Do you think?
Thank you for update!
I think that your code is more concise than mine.
There seems to be no problem.

Regards,
Koyu Tanigawa

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2021-10-04 03:56:59 Re: Triage on old commitfest entries
Previous Message Masahiko Sawada 2021-10-04 00:31:06 Re: Skipping logical replication transactions on subscriber side