]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Minor fixes for test_bitmapset.c
authorDavid Rowley <drowley@postgresql.org>
Sun, 19 Apr 2026 21:58:40 +0000 (09:58 +1200)
committerDavid Rowley <drowley@postgresql.org>
Sun, 19 Apr 2026 21:58:40 +0000 (09:58 +1200)
commit5142f0093e648d1a32fdcc7c835d17fa103e1239
tree3131f1563c77bec992033bf3e3874e7acf69dff7
parent9018c7d37bb464cd53567c0b553a6f49b50bec78
Minor fixes for test_bitmapset.c

1. Make it so test_random_operations() can accept a NULL to have the
   function select a random seed.
2. Widen the seed parameter of test_random_operations() to bigint.
   Without that, it'll be impossible to run the function with a seed
   which was selected by GetCurrentTimestamp(), and if a randomly
   selected seed ever results in a failure, we'll likely want to run
   with the same seed to debug the issue.
3. Report the seed in the error messages in test_random_operations().
   If the buildfarm were ever to fail there, we'd certainly want to know
   what this was.
4. Add CHECK_FOR_INTERRUPTS() to test_random_operations().  Someone might
   run with a large num_ops and they'd have no way to cancel the query.
5. Minor cosmetic fixes; header order and whitespace issue.

To allow #1, the STRICT modifier had to be removed.  The additional
prechecks were added as I didn't see how else to handle someone passing
those parameters as NULL.

Author: David Rowley <dgrowleyml@gmail.com>
Reviewed-by: Greg Burd <greg@burd.me>
Discussion: https://postgr.es/m/CAApHDvrDW9W72vAr7h7XeCu7+Qz-_Vff02Q+RPPuVeM0Qf0MCw@mail.gmail.com
src/test/modules/test_bitmapset/expected/test_bitmapset.out
src/test/modules/test_bitmapset/sql/test_bitmapset.sql
src/test/modules/test_bitmapset/test_bitmapset--1.0.sql
src/test/modules/test_bitmapset/test_bitmapset.c