Fix BackgroundPsql's set_query_timer_restart() issue without argument.
authorMasahiko Sawada <msawada@postgresql.org>
Thu, 30 Nov 2023 01:14:17 +0000 (10:14 +0900)
committerMasahiko Sawada <msawada@postgresql.org>
Thu, 30 Nov 2023 01:14:17 +0000 (10:14 +0900)
The set_query_timer_restart() required an argument to define a value
to query_timer_restart, but none of the existing callers passes an
argument to this function.

This changes the function to set a value without an argument.

Backpatch through 16 where the background psql TAP functions were
refactored by 664d757531e1.

Reviewed-by: Bharath Rupireddy, Tom Lane
Discussion: https://postgr.es/m/CAD21AoA0B6VKe_5A9nZi8i5umwSN-zJJuPVNht9DaOZ9SJumMA@mail.gmail.com
Backpatch-through: 16

src/test/perl/PostgreSQL/Test/BackgroundPsql.pm

index 924b57ab2186261346e8ab7402d5c2622ca70605..764f0a538017d13b7fccad105210635d9ebf35de 100644 (file)
@@ -301,7 +301,7 @@ sub set_query_timer_restart
 {
        my $self = shift;
 
-       $self->{query_timer_restart} = shift if @_;
+       $self->{query_timer_restart} = 1;
        return $self->{query_timer_restart};
 }