diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-10-16 12:32:28 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-10-16 12:32:28 -0400 |
| commit | 94ccffb2b5b22408f0f5395b820bcfa5c1b172d3 (patch) | |
| tree | c4a9d1988d321d029b66df3166e591ce42d2cb25 | |
| parent | 4f6a77e46af26eeeecfd75d50eabd090eb99f3c8 (diff) | |
| download | postgresql-94ccffb2b5b22408f0f5395b820bcfa5c1b172d3.tar.gz postgresql-94ccffb2b5b22408f0f5395b820bcfa5c1b172d3.zip | |
Make PostgresNode.pm's poll_query_until() more chatty about failures.
Reporting only the stderr is unhelpful when the problem is that the
server output we're getting doesn't match what was expected. So we
should report the query output too; and just for good measure, let's
print the query we used and the output we expected.
Back-patch to 9.5 where poll_query_until was introduced.
Discussion: https://postgr.es/m/17913.1539634756@sss.pgh.pa.us
| -rw-r--r-- | src/bin/pg_rewind/RewindTest.pm | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/src/bin/pg_rewind/RewindTest.pm b/src/bin/pg_rewind/RewindTest.pm index 426bd6e5cf3..734adb72ec5 100644 --- a/src/bin/pg_rewind/RewindTest.pm +++ b/src/bin/pg_rewind/RewindTest.pm @@ -151,9 +151,18 @@ sub poll_query_until $attempts++; } - # The query result didn't change in 90 seconds. Give up. Print the stderr - # from the last attempt, hopefully that's useful for debugging. - diag $stderr; + # The query result didn't change in 90 seconds. Give up. Print the + # output from the last attempt, hopefully that's useful for debugging. + chomp($stderr); + $stderr =~ s/\r//g if $Config{osname} eq 'msys'; + diag qq(poll_query_until timed out executing this query: +$query +expecting this output: +t +last actual query output: +$stdout +with stderr: +$stderr); return 0; } |