Re: pgsql: Improve BRIN minmax-multi opclass test coverage - Mailing list pgsql-committers

From Peter Eisentraut
Subject Re: pgsql: Improve BRIN minmax-multi opclass test coverage
Date
Msg-id a35eaf0f-b19c-7797-e296-7b7e2adc473e@eisentraut.org
Whole thread Raw
In response to pgsql: Improve BRIN minmax-multi opclass test coverage  (Tomas Vondra <tomas.vondra@postgresql.org>)
Responses Re: pgsql: Improve BRIN minmax-multi opclass test coverage
List pgsql-committers
These new tests introduced a few new calls of the md5() function.  This 
should be avoided (see commit 208bf364a9).  You can replace these easily 
with the fipshash() function instead, but I think you then also need to 
change some of the constants in the tests.  Could you look at this again?


On 02.07.23 10:34, Tomas Vondra wrote:
> Improve BRIN minmax-multi opclass test coverage
> 
> Per the code coverage report, the existing regression tests did not
> exercice some a couple important BRIN minmax-multi code paths.
> 
> - The tests focused on testing planning with a range of scan key
>    strategies, but not the execution. Fixed by adding queries that
>    actually test query execution for both equality and inequality.
> 
> - All tests created indexes after inserting data, but this only
>    exercises the CREATE INDEX strategy that sees all values at once, not
>    incremental summary updates. The new tests flip the order and create
>    the index before adding data.
> 
> - The assert check(s) validating correctness of expanded ranges were
>    present only in the "union" code path, which is not covered by
>    regression tests at all (as it requires concurrency etc.). Fixed by
>    adding the asserts to a couple more places.
> 
> Reviewed-by: Heikki Linnakangas
> Discussion: https://postgr.es/m/57020b2e-d9c9-9bc7-4892-b36d9bb07563%40enterprisedb.com
> 
> Branch
> ------
> master
> 
> Details
> -------
> https://git.postgresql.org/pg/commitdiff/0457109344b46f481f9bf09b85674695ba57c8e4
> 
> Modified Files
> --------------
> src/backend/access/brin/brin_minmax_multi.c |   9 +
> src/test/regress/expected/brin_multi.out    | 357 ++++++++++++++++++++++++++++
> src/test/regress/sql/brin_multi.sql         | 165 +++++++++++++
> 3 files changed, 531 insertions(+)
> 




pgsql-committers by date:

Previous
From: Amit Kapila
Date:
Subject: pgsql: Simplify the logical worker type checks by using the switch on w
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Add list of acknowledgments to release notes