pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  

help / color / mirror / Atom feed
From: Alexander Lakhin <exclusion@gmail.com>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: pgsql-bugs@lists.postgresql.org
Subject: Re: BUG #17368: Assert failed in GetSafeSnapshot() for SERIALIZABLE READ ONLY DEFERRABLE transaction
Date: Thu, 9 Mar 2023 12:00:00 +0300
Message-ID: <77a67fa9-9c69-d1e8-2b83-15b27e11ccf7@gmail.com> (raw)
In-Reply-To: <CA+hUKG+Y96oENFj_boGCw6cBiUhM5wA=kiNReodZFLmsBtfwRQ@mail.gmail.com>
References: <17368-98a4f99e8e4b4402@postgresql.org>
	<18a116d2-7927-6a62-aa60-ab16c53183f9@gmail.com>
	<CA+hUKG+4m+fvnM_Czyp3Z_h4SaqmwR7sq9XE=k0GcaL_afEfUA@mail.gmail.com>
	<CA+hUKGKYLSa4S7cMYgZXXA+cNTdr1COWtNj1HSc-Y62jwWmv7w@mail.gmail.com>
	<CA+hUKGKT1RCYcy_p8RKGs0C5-8c2=J38xMOcYTxsPL=x6sr-cA@mail.gmail.com>
	<33d30d9c-9491-386e-d19f-b58861683fe2@gmail.com>
	<CA+hUKGKRkLpPUEECYwdp8s8TPeDfzwTLi7Q_-H9t+RCGC6_ArA@mail.gmail.com>
	<CA+hUKG+Y96oENFj_boGCw6cBiUhM5wA=kiNReodZFLmsBtfwRQ@mail.gmail.com>

09.03.2023 07:39, Thomas Munro wrote:
> Pushed, without that test.
>
> I realised that the test would not be stable in the build farm.  If we
> made the lock timeout high, the test would be slow, but if we made it
> low, then there would be two possible outputs depending on a race, and
> 10ms as you had it seems -- I guess? -- likely to be unstable under
> valgrind or an RPi2 or something.  There is probably some clever way
> to write a different test schedule, but the new code is exercised by
> existing tests, and the assertion has been failing once every couple
> of days on CI since I started collecting that data a few weeks ago, so
> we have some kind of coverage, at least for master.

I had thought that we can use the same timeout that can be seen in
a couple of other tests, but now I see that those tests don't depend
on it directly/have outweighing timeouts, so I completely agree, that
the test proposed is not elaborated enough to be committed.

Thank you!

Best regards,
Alexander





view thread (12+ messages)

Message-ID: <77a67fa9-9c69-d1e8-2b83-15b27e11ccf7@gmail.com>
Permalink:  ../77a67fa9-9c69-d1e8-2b83-15b27e11ccf7@gmail.com/
Also on:    postgresql.org/message-id/77a67fa9-9c69-d1e8-2b83-15b27e11ccf7@gmail.com

 · 

reply
Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: exclusion@gmail.com, thomas.munro@gmail.com, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #17368: Assert failed in GetSafeSnapshot() for SERIALIZABLE READ ONLY DEFERRABLE transaction
  In-Reply-To: <77a67fa9-9c69-d1e8-2b83-15b27e11ccf7@gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox