From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Fix pattern matching logic for logs in TAP tests of pgbench |
Date: | 2021-06-24 21:54:51 |
Message-ID: | E1lwXJD-0005Ku-LB@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Fix pattern matching logic for logs in TAP tests of pgbench
The logic checking for the format of per-thread logs used grep() with
directly "$re", which would cause the test to consider all the logs as
a match without caring about their format at all. Using "/$re/" makes
grep() perform a regex test, which is what we want here.
While on it, improve some of the tests to be more picky with the
patterns expected and add more comments to describe the tests.
Issue discovered while digging into a separate patch.
Author: Fabien Coelho, Michael Paquier
Discussion: https://postgr.es/m/YNPsPAUoVDCpPOGk@paquier.xyz
Backpatch-through: 11
Branch
------
REL_11_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/0efd2a1a66dd6e27b097ad533311d8e1ad7ca309
Modified Files
--------------
src/bin/pgbench/t/001_pgbench_with_server.pl | 25 +++++++++++++++----------
1 file changed, 15 insertions(+), 10 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Munro | 2021-06-24 22:06:23 | Re: Ubuntu 14.04 (trusty) Postgres 13 deb package |
Previous Message | Bhavesh Mistry | 2021-06-24 21:35:22 | Ubuntu 14.04 (trusty) Postgres 13 deb package |