Re: reducing isolation tests runtime
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andres Freund <andres@anarazel.de>
Cc: Alvaro Herrera <alvherre@alvh.no-ip.org>,
Thomas Munro <thomas.munro@enterprisedb.com>,
pgsql-hackers@postgresql.org
Date: 2019-02-13T14:34:35Z
Lists: pgsql-hackers
Andres Freund <andres@anarazel.de> writes: > I'm working an updated version of this. Adding the new tests is a bit > painful because of conflicting names making it harder than necessary to > schedule tests. While it's possible to work out a schedule that doesn't > conflict, it's pretty annoying to do and more importantly seems fragile > - it's very easy to create schedules that succeed on one machine, and > not on another, based on how slow which tests are. > I'm more inclined to be a bit more aggressive in renaming tables - > there's not much point in having a lot of "foo"s around. So I'm > inclined to rename some of the names that are more likely to > conflict. If we agree on doing that, I'd like to do that first, and > commit that more aggressively than the schedule itself. +1 > Do we want to maintain a serial version of the schedule too? Some of the slower buildfarm critters use MAX_CONNECTIONS to limit the load on their hosts. As long as the isolation tests honor that, I don't see a real need for a separate serial schedule. (We've talked about retiring the serial sched for the main regression tests, and while that trigger's not been pulled yet, I think it's just a matter of time. So making the isolation tests follow that precedent seems wrong anyway.) regards, tom lane
Commits
-
Update obsolete sentence in README.parallel.
- 28e04155f17c 11.0 cited
-
Rewrite ConditionVariableBroadcast() to avoid live-lock.
- aced5a92bf46 11.0 cited
-
Tweak parallel hash join test case in hopes of improving stability.
- 934c7986f4a0 11.0 landed
-
Rename pg_rewind's copy_file_range() to avoid conflict with new linux syscall.
- 3e68686e2c55 11.0 cited
-
Fix some minor errors in new PHJ code.
- 6fcde2406304 11.0 landed
-
Fix EXPLAIN ANALYZE output for Parallel Hash.
- 93ea78b17c47 11.0 landed
-
Fix rare assertion failure in parallel hash join.
- f83040c62a78 11.0 landed
-
Cancel CV sleep during subtransaction abort.
- f3decdc94ea3 10.2 landed
- 59d1e2b95a82 11.0 landed
-
Add parallel-aware hash joins.
- 1804284042e6 11.0 cited
-
Fix EXPLAIN ANALYZE of hash join when the leader doesn't participate.
- 5bcf389ecfd4 11.0 cited
-
Add some regression tests that exercise hash join code.
- fa330f9adf4e 11.0 cited