Fix the random test failure in 001_rep_changes.
authorAmit Kapila
Wed, 1 Sep 2021 04:30:12 +0000 (10:00 +0530)
committerAmit Kapila
Wed, 1 Sep 2021 04:30:12 +0000 (10:00 +0530)
commitb7ad093d50e13aadfffb1662f53cd16a1c59e09d
treee8acd6aee70befed0863adcc963364e02f3d1ddc
parent0d892cf73a13b3a32af438a059a168e711aa0a7f
Fix the random test failure in 001_rep_changes.

The check to test whether the subscription workers were restarting after a
change in the subscription was failing. The reason was that the test was
assuming the walsender started before it reaches the 'streaming' state and
the walsender was exiting due to an error before that. Now, the walsender
was erroring out before reaching the 'streaming' state because it tries to
acquire the slot before the previous walsender has exited.

In passing, improve the die messages so that it is easier to investigate
the failures in the future if any.

Reported-by: Michael Paquier, as per buildfarm
Author: Ajin Cherian
Reviewed-by: Masahiko Sawada, Amit Kapila
Backpatch-through: 10, where this test was introduced
Discussion: https://postgr.es/m/[email protected]
src/test/subscription/t/001_rep_changes.pl