address comments

apple · Apr 29, 2024 · bb8ba15 · bb8ba15
1 parent 053546c
commit bb8ba15
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/fdbclient/ClientKnobs.cpp b/fdbclient/ClientKnobs.cpp
@@ -269,7 +269,7 @@ void ClientKnobs::initialize(Randomize randomize) {
 
 	init( CONSISTENCY_CHECK_RATE_LIMIT_MAX,        50e6 ); // Limit in per sec
 	init( CONSISTENCY_CHECK_ONE_ROUND_TARGET_COMPLETION_TIME,	7 * 24 * 60 * 60 ); // 7 days
-	init( CONSISTENCY_CHECK_URGENT_NEXT_WAIT_TIME,    600 ); if( randomize && BUGGIFY ) CONSISTENCY_CHECK_URGENT_NEXT_WAIT_TIME = 10;
+	init( CONSISTENCY_CHECK_URGENT_NEXT_WAIT_TIME,         600 );
 	init( CONSISTENCY_CHECK_URGENT_BATCH_SHARD_COUNT,       10 ); if( randomize && BUGGIFY ) CONSISTENCY_CHECK_URGENT_BATCH_SHARD_COUNT = 2;
 	init( CONSISTENCY_CHECK_URGENT_RETRY_DEPTH_MAX,         10 ); if( randomize && BUGGIFY ) CONSISTENCY_CHECK_URGENT_RETRY_DEPTH_MAX = 1;
 	init( CONSISTENCY_CHECK_URGENT_RANGE_BEGIN_0,           "" ); if( randomize && BUGGIFY ) CONSISTENCY_CHECK_URGENT_RANGE_BEGIN_0 = "";

diff --git a/fdbserver/tester.actor.cpp b/fdbserver/tester.actor.cpp
@@ -1915,10 +1915,10 @@ ACTOR Future<Void> runConsistencyCheckerUrgentHolder(Reference<AsyncVar<Optional
                                                      bool repeatRun) {
 	loop {
 		wait(runConsistencyCheckerUrgentCore(cc, cx, testers, minTestersExpected));
-		wait(delay(CLIENT_KNOBS->CONSISTENCY_CHECK_URGENT_NEXT_WAIT_TIME)); // wait for 10 minutes
 		if (!repeatRun) {
 			break;
 		}
+		wait(delay(CLIENT_KNOBS->CONSISTENCY_CHECK_URGENT_NEXT_WAIT_TIME)); // wait for 10 minutes
 	}
 	return Void();
 }