Skip to content

Commit b6fae2e

Browse files
committed
Ensure testAckedIndexing uses disruption index settings
AbstractDisruptionTestCase set a lower global checkpoint sync interval setting, but this was ignored by testAckedIndexing, which has led to spurious test failures Relates #41068, #38931
1 parent badfcdf commit b6fae2e

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

server/src/main/java/org/elasticsearch/indices/recovery/PeerRecoveryTargetService.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -359,7 +359,7 @@ private StartRecoveryRequest getStartRecoveryRequest(final RecoveryTarget recove
359359
logger.trace("{} preparing for file-based recovery from [{}]", recoveryTarget.shardId(), recoveryTarget.sourceNode());
360360
} else {
361361
logger.trace(
362-
"{} preparing for sequence-number-based recovery starting at local checkpoint [{}] from [{}]",
362+
"{} preparing for sequence-number-based recovery starting at sequence number [{}] from [{}]",
363363
recoveryTarget.shardId(),
364364
startingSeqNo,
365365
recoveryTarget.sourceNode());

server/src/test/java/org/elasticsearch/discovery/ClusterDisruptionIT.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ public void testAckedIndexing() throws Exception {
115115

116116
assertAcked(prepareCreate("test")
117117
.setSettings(Settings.builder()
118+
.put(indexSettings())
118119
.put(IndexMetaData.SETTING_NUMBER_OF_SHARDS, 1 + randomInt(2))
119120
.put(IndexMetaData.SETTING_NUMBER_OF_REPLICAS, randomInt(2))
120121
));

0 commit comments

Comments
 (0)