Re: Add “FOR UPDATE NOWAIT” lock details to the log.

From: Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>
To: Yuki Seino <seinoyu(at)oss(dot)nttdata(dot)com>, Jelte Fennema-Nio <postgres(at)jeltef(dot)nl>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Add “FOR UPDATE NOWAIT” lock details to the log.
Date: 2025-02-28 12:51:10
Message-ID: c492a261-f813-4a4f-84de-6f75e7e3c74c@oss.nttdata.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2025/02/28 21:08, Yuki Seino wrote:
>
> On 2025/02/27 15:44, Yuki Seino wrote:
>> (4) https://github.com/postgres/postgres/blob/master/src/backend/access/heap/heapam_handler.c#L463
>> ...I don't know how to reproduce it.
> I have confirmed that (4) can be reproduced using the following procedure.
>
> (4) https://github.com/postgres/postgres/blob/master/src/backend/access/heap/heapam_handler.c#L463
> tx1=# SELECT pg_advisory_lock(0);
> tx2=# SELECT * FROM pgbench_accounts WHERE pg_advisory_lock(0) IS NOT NULL FOR UPDATE NOWAIT;
> tx1=# UPDATE pgbench_accounts SET aid = aid WHERE aid = '1';
> tx1=# BEGIN;
> tx1=# UPDATE pgbench_accounts SET aid = aid WHERE aid = '1';
> tx1=# SELECT pg_advisory_unlock(0);
>
> Send the modified patch.

Thanks for updating the patch!

I encountered a compilation error with the patch. You can also see the error in the patch tester.
https://cirrus-ci.com/task/5070779370438656

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 Yura Sokolov 2025-02-28 13:03:33 Re: Implement waiting for wal lsn replay: reloaded
Previous Message Shubham Khanna 2025-02-28 12:41:19 Re: Enhance 'pg_createsubscriber' to retrieve databases automatically when no database is provided.