pgsql: Reduce length of GIN predicate locking isolation test suite

From: Alexander Korotkov <akorotkov(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Reduce length of GIN predicate locking isolation test suite
Date: 2018-12-28 00:44:26
Message-ID: E1gcgGI-0007iD-HT@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Reduce length of GIN predicate locking isolation test suite

Isolation test suite of GIN predicate locking was criticized for being too slow,
especially under Valgrind. This commit is intended to accelerate it. Tests are
simplified in the following ways.

1) Amount of data is reduced. We're now close to the minimal amount of data,
which produces at least one posting tree and at least two pages of entry
tree.
2) Three isolation tests are merged into one.
3) Only one tuple is queried from posting tree. So, locking of index is the
same, but tuple locks are not propagated to relation lock. Also, it is
faster.
4) Test cases itself are simplified. Now each test case run just one INSERT
and one SELECT involving GIN, which either conflict or not.

Discussion: https://postgr.es/m/20181204000740.ok2q53nvkftwu43a%40alap3.anarazel.de
Reported-by: Andres Freund
Tested-by: Andrew Dunstan
Author: Alexander Korotkov
Backpatch-through: 11

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/0c6f4f9212cf3155b258910acf80fec89f49b767

Modified Files
--------------
.../expected/predicate-gin-fastupdate.out | 30 -
.../isolation/expected/predicate-gin-nomatch.out | 15 -
src/test/isolation/expected/predicate-gin.out | 783 +++++++--------------
src/test/isolation/isolation_schedule | 2 -
.../isolation/specs/predicate-gin-fastupdate.spec | 49 --
.../isolation/specs/predicate-gin-nomatch.spec | 35 -
src/test/isolation/specs/predicate-gin.spec | 174 ++---
7 files changed, 320 insertions(+), 768 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Michael Paquier 2018-12-28 01:20:46 pgsql: Clarify referential actions in docs of CREATE/ALTER TABLE
Previous Message Alexander Korotkov 2018-12-28 00:44:15 pgsql: Reduce length of GIN predicate locking isolation test suite