Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit0a576cd

Browse files
committed
Make PostgresNode.pm's poll_query_until() more chatty about failures.
Reporting only the stderr is unhelpful when the problem is that theserver output we're getting doesn't match what was expected. So weshould report the query output too; and just for good measure, let'sprint 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
1 parentafb5fb2 commit0a576cd

File tree

1 file changed

+12
-3
lines changed

1 file changed

+12
-3
lines changed

‎src/test/perl/PostgresNode.pm

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1257,9 +1257,18 @@ sub poll_query_until
12571257
$attempts++;
12581258
}
12591259

1260-
# The query result didn't change in 180 seconds. Give up. Print the stderr
1261-
# from the last attempt, hopefully that's useful for debugging.
1262-
diag$stderr;
1260+
# The query result didn't change in 180 seconds. Give up. Print the
1261+
# output from the last attempt, hopefully that's useful for debugging.
1262+
chomp($stderr);
1263+
$stderr =~s/\r//gif$TestLib::windows_os;
1264+
diagqq(poll_query_until timed out executing this query:
1265+
$query
1266+
expecting this output:
1267+
$expected
1268+
last actual query output:
1269+
$stdout
1270+
with stderr:
1271+
$stderr);
12631272
return 0;
12641273
}
12651274

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp