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

Commit1db439a

Browse files
committed
Raise some timeouts to 180s, in test code.
Slow runs of buildfarm members chipmunk, hornet and mandrill saw theshorter timeouts expire. The 180s timeout in poll_query_until has beentrouble-free since2a0f89c introducedit two years ago, so use 180s more widely. Back-patch to 9.6, where thefirst of these timeouts was introduced.Reviewed by Michael Paquier.Discussion:https://postgr.es/m/20181209001601.GC2973271@rfd.leadboat.com
1 parent001bb9f commit1db439a

File tree

4 files changed

+13
-13
lines changed

4 files changed

+13
-13
lines changed

‎src/test/isolation/README

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,8 +108,8 @@ Each step may contain commands that block until further action has been taken
108108
deadlock). A test that uses this ability must manually specify valid
109109
permutations, i.e. those that would not expect a blocked session to execute a
110110
command. If a test fails to follow that rule, isolationtester will cancel it
111-
after60 seconds. If the cancel doesn't work, isolationtester will exit
112-
uncleanly after a total of75 seconds of wait time. Testing invalid
111+
after180 seconds. If the cancel doesn't work, isolationtester will exit
112+
uncleanly after a total of200 seconds of wait time. Testing invalid
113113
permutations should be avoided because they can make the isolation tests take
114114
a very long time to run, and they serve no useful testing purpose.
115115

‎src/test/isolation/isolationtester.c

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -783,15 +783,15 @@ try_complete_step(Step *step, int flags)
783783
td+= (int64)current_time.tv_usec- (int64)start_time.tv_usec;
784784

785785
/*
786-
* After60 seconds, try to cancel the query.
786+
* After180 seconds, try to cancel the query.
787787
*
788788
* If the user tries to test an invalid permutation, we don't want
789789
* to hang forever, especially when this is running in the
790-
* buildfarm.So try to cancel it after a minute. This will
791-
*presumably lead to this permutation failing, but remaining
792-
*permutations and tests should still beOK.
790+
* buildfarm.This will presumably lead to this permutation
791+
*failing, but remaining permutations and tests should still be
792+
* OK.
793793
*/
794-
if (td>60*USECS_PER_SEC&& !canceled)
794+
if (td>180*USECS_PER_SEC&& !canceled)
795795
{
796796
PGcancel*cancel=PQgetCancel(conn);
797797

@@ -808,15 +808,15 @@ try_complete_step(Step *step, int flags)
808808
}
809809

810810
/*
811-
* After75 seconds, just give up and die.
811+
* After200 seconds, just give up and die.
812812
*
813813
* Since cleanup steps won't be run in this case, this may cause
814814
* later tests to fail. That stinks, but it's better than waiting
815815
* forever for the server to respond to the cancel.
816816
*/
817-
if (td>75*USECS_PER_SEC)
817+
if (td>200*USECS_PER_SEC)
818818
{
819-
fprintf(stderr,"step %s timed out after75 seconds\n",
819+
fprintf(stderr,"step %s timed out after200 seconds\n",
820820
step->name);
821821
exit_nicely();
822822
}

‎src/test/recovery/t/006_logical_decoding.pl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@
7272
print"waiting to replay$endpos\n";
7373

7474
my$stdout_recv =$node_master->pg_recvlogical_upto(
75-
'postgres','test_slot',$endpos,10,
75+
'postgres','test_slot',$endpos,180,
7676
'include-xids'=>'0',
7777
'skip-empty-xacts'=>'1');
7878
chomp($stdout_recv);
@@ -84,7 +84,7 @@
8484
)ordie"slot never became inactive";
8585

8686
$stdout_recv =$node_master->pg_recvlogical_upto(
87-
'postgres','test_slot',$endpos,10,
87+
'postgres','test_slot',$endpos,180,
8888
'include-xids'=>'0',
8989
'skip-empty-xacts'=>'1');
9090
chomp($stdout_recv);

‎src/test/recovery/t/010_logical_decoding_timelines.pl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,7 @@ BEGIN
183183

184184
$stdout =$node_replica->pg_recvlogical_upto(
185185
'postgres','before_basebackup',
186-
$endpos,30,
186+
$endpos,180,
187187
'include-xids'=>'0',
188188
'skip-empty-xacts'=>'1');
189189

0 commit comments

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