Re: old_snapshot: add test for coverage - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: old_snapshot: add test for coverage
Date
Msg-id C3CB0C0A-3B59-488E-8B8D-8A158990AC9D@yesql.se
Whole thread Raw
In response to Re: old_snapshot: add test for coverage  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
> On 8 Aug 2022, at 07:37, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Dong Wook Lee <sh95119@gmail.com> writes:

>> I wrote a test of the old_snapshot extension for coverage.
> 
> Hmm, does this really provide any meaningful coverage?  The test
> sure looks like it's not doing much.

Looking at this I agree, this test doesn't provide enough to be of value and
the LIMIT 0 might even hide bugs under a postive test result.  I think we
should mark this entry RwF.

--
Daniel Gustafsson        https://vmware.com/




pgsql-hackers by date:

Previous
From: Spyridon Dimitrios Agathos
Date:
Subject: CREATE UNLOGGED TABLE seq faults when debug_discard_caches=1
Next
From: Bharath Rupireddy
Date:
Subject: Introduce a new view for checkpointer related stats