>> My hoary animal prairiedog doesn't like this [1]: >> >> # Failed test 'concurrent update with retrying stderr /(?s-xim:client >> (0|1) got an error in command 3 \\(SQL\\) of script 0; ERROR: could not >> serialize access due to concurrent update\\b.*\\g1)/' >> # at t/001_pgbench_with_server.pl line 1229. >> # 'pgbench: pghost: /tmp/nhghgwAoki pgport: 58259 >> nclients: 2 nxacts: 1 dbName: postgres >> ... >> # pgbench: client 0 got an error in command 3 (SQL) of script 0; ERROR: >> could not serialize access due to concurrent update >> ... >> # ' >> # doesn't match '(?s-xim:client (0|1) got an error in command 3 >> \\(SQL\\) of script 0; ERROR: could not serialize access due to concurrent >> update\\b.*\\g1)' >> # Looks like you failed 1 test of 425. >> >> I'm not sure what the "\\b.*\\g1" part of this regex is meant to >> accomplish, but it seems to be assuming more than it should >> about the output format of TAP messages. > > I have edited the test code from the original patch by mistake, but > I could not realize because the test works in my machine without any > errors somehow. > > I attached a patch to fix the test as was in the original patch, where > backreferences are used to check retry of the same query.
My machine (Ubuntu 20) did not complain either. Maybe perl version difference? Any way, the fix pushed. Let's see how prairiedog feels. Best reagards, -- Tatsuo Ishii SRA OSS, Inc. Japan English: http://www.sraoss.co.jp/index_en.php Japanese:http://www.sraoss.co.jp
