> On 28 Jul 2026, at 15:13, Daniel Gustafsson <[email protected]> wrote: >> On 28 Jul 2026, at 12:01, Zsolt Parragi <[email protected]> wrote:
>> I attached a patch that adds testcases for these and fixes the issues.
>
> Thanks, I've reviewed and tested your patch and intend to push it along with
> your other earlier patch shortly.
BF member porpoise points out that we seems to have missed a detach in the
injection points test:
# die: error running SQL: 'psql:<stdin>:1: ERROR: injection point
"datachecksumsworker-fake-temptable-wait" already defined'
# while running 'psql --no-psqlrc --no-align --tuples-only --quiet --dbname
port=19905 host=/tmp/oQlRAvFxKv dbname='postgres' --file - --variable
ON_ERROR_STOP=1' with sql 'SELECT
injection_points_attach('datachecksumsworker-fake-temptable-wait','wait');' at
/home/buildfarm/pg-buildfarm/builds/HEAD/pgsql/src/test/modules/test_checksums/t/005_injection.pl
line 92.
# Looks like your test exited with 29 just after 8.
https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=porpoise&dt=2026-07-30%2007%3A22%3A50&stg=test_checksums-check
It only fails when running the extended tests which is why it didn't fail in CI
or on more BF members (and I clearly forgot to run those tests before pushing).
Does this seem correct to you?
--
Daniel Gustafsson
0001-Make-sure-to-detach-injection-points-for-re-attachin.patch
Description: Binary data
