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

From Tom Lane
Subject Re: old_snapshot: add test for coverage
Date
Msg-id 319829.1659937048@sss.pgh.pa.us
Whole thread Raw
In response to old_snapshot: add test for coverage  (Dong Wook Lee <sh95119@gmail.com>)
Responses Re: old_snapshot: add test for coverage
Re: old_snapshot: add test for coverage
List pgsql-hackers
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.

I spent some time a week or so ago trying to graft testing of
contrib/old_snapshot into src/test/modules/snapshot_too_old.
I didn't come up with anything I liked, but I still think
that that might lead to a more thorough test than a standalone
exercise.

            regards, tom lane



pgsql-hackers by date:

Previous
From: vignesh C
Date:
Subject: Re: Handle infinite recursion in logical replication setup
Next
From: Dilip Kumar
Date:
Subject: Re: Perform streaming logical transactions by background workers and parallel apply