Adding a test for speculative insert abort case - Mailing list pgsql-hackers

From Melanie Plageman
Subject Adding a test for speculative insert abort case
Date
Msg-id CAAKRu_a7hbyrk=wveHYhr4LbcRnRCG=yPUVoQYB9YO1CdUBE9Q@mail.gmail.com
Whole thread Raw
Responses Re: Adding a test for speculative insert abort case  (Andres Freund <andres@anarazel.de>)
Re: Adding a test for speculative insert abort case  (Peter Geoghegan <pg@bowt.ie>)
Re: Adding a test for speculative insert abort case  (Thomas Munro <thomas.munro@gmail.com>)
List pgsql-hackers
Today, while poking around the table_complete_speculative code which Ashwin
mentioned in [1], we were trying to understand when exactly we would complete a
speculative insert by aborting.

We added a logging message to heapam_tuple_complete_speculative before it calls
heap_abort_speculative and ran the regression and isolation tests to see what
test failed so we knew how to exercise this codepath.
No tests failed, so we spent some time trying to understand when 'succeeded'
would be true coming into heap_tuple_complete_speculative.

Eventually, we figured out that if one transaction speculatively inserts a
tuple into a table with a unique index and then pauses before inserting the
value into the index, and while it is paused, another transaction successfully
inserts a value which would conflict with that value, it would result in an
aborted speculative insertion.

t1(id,val)
unique index t1(id)

s1: insert into t1 values(1, 'someval') on conflict(id) do update set val = 'someotherval';
s1: pause in ExecInsert before calling ExecInsertIndexTuples
s2: insert into t1 values(1, 'someval');
s2: continue

We don't know of a way to add this scenario to the current isolation framework.

Can anyone think of a good way to put this codepath under test?

- Melanie & Ashwin

[1] https://www.postgresql.org/message-id/CALfoeitk7-TACwYv3hCw45FNPjkA86RfXg4iQ5kAOPhR%2BF1Y4w%40mail.gmail.com

pgsql-hackers by date:

Previous
From: Justin Pryzby
Date:
Subject: Re: doc: improve PG 12 to_timestamp()/to_date() wording
Next
From: Andres Freund
Date:
Subject: Re: Adding a test for speculative insert abort case