From 6828b03b062bf9eee7b5e5e7ee1cedd4e174ab29 Mon Sep 17 00:00:00 2001 From: Michael Paquier Date: Wed, 12 Aug 2026 16:23:04 +0900 Subject: [PATCH] Fix poll_query_until() timeout handling with an undefined query in Cluster.pm On timeout, poll_query_until() would fail to print its report when "$query" was undefined. Instead of the expected report, this prevented the contents of stdout and stderr from being shown. Note that some of the in-core tests use an undefined query for connection-only polls. Author: Bryan Green Reviewed-by: Jonathan Gonzalez V. Discussion: https://postgr.es/m/0f515c6d-6032-4c5f-80ac-5c78faae9522@gmail.com Backpatch-through: 14 --- src/test/perl/PostgreSQL/Test/Cluster.pm | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/test/perl/PostgreSQL/Test/Cluster.pm b/src/test/perl/PostgreSQL/Test/Cluster.pm index 3eae4cf6281..920d831be9e 100644 --- a/src/test/perl/PostgreSQL/Test/Cluster.pm +++ b/src/test/perl/PostgreSQL/Test/Cluster.pm @@ -2808,8 +2808,10 @@ sub poll_query_until # Give up. Print the output from the last attempt, hopefully that's useful # for debugging. + my $msg_query = $query; + $msg_query = '(connection attempt only)' unless defined $query; diag qq(poll_query_until timed out executing this query: -$query +$msg_query expecting this output: $expected last actual query output: