Re: Cutting test runtime for src/test/modules/snapshot_too_old

2022-08-02 Thread Tom Lane
Robert Haas writes: > Yeah, I feel like it was a mistake to allow the list of permutations > to be unspecified. It encourages people to just run them all, which is > almost never a thoughtful decision. Maybe there's something to be said > for running these tests in one successful permutation and o

Re: Cutting test runtime for src/test/modules/snapshot_too_old

2022-08-02 Thread Robert Haas
On Tue, Aug 2, 2022 at 11:38 AM Tom Lane wrote: > I've complained before that the snapshot_too_old TAP tests seem > ridiculously slow --- close to a minute of runtime even on very fast > machines. Today I happened to look closer and realized that there's > an absolutely trivial way to cut that.

Cutting test runtime for src/test/modules/snapshot_too_old

2022-08-02 Thread Tom Lane
I've complained before that the snapshot_too_old TAP tests seem ridiculously slow --- close to a minute of runtime even on very fast machines. Today I happened to look closer and realized that there's an absolutely trivial way to cut that. The core of the slow runtime is that there's a "pg_sleep(