Skip to content

Commit e704d2a

Browse files
register no-op settings consumer so that settings validations works properly
1 parent 5888a5d commit e704d2a

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

server/src/main/java/org/elasticsearch/index/engine/ThreadPoolMergeExecutorService.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -196,6 +196,11 @@ public Iterator<Setting<?>> settings() {
196196
if (clusterSettings.get(USE_THREAD_POOL_MERGE_SCHEDULER_SETTING)) {
197197
return new ThreadPoolMergeExecutorService(threadPool, clusterSettings, nodeEnvironment);
198198
} else {
199+
// register no-op settings consumer so that settings validations works properly,
200+
// i.e. to reject watermark and max headroom updates if the thread pool merge scheduler is disabled
201+
clusterSettings.addSettingsUpdateConsumer(INDICES_MERGE_DISK_HIGH_WATERMARK_SETTING, (ignored) -> {});
202+
clusterSettings.addSettingsUpdateConsumer(INDICES_MERGE_DISK_HIGH_MAX_HEADROOM_SETTING, (ignored) -> {});
203+
clusterSettings.addSettingsUpdateConsumer(INDICES_MERGE_DISK_CHECK_INTERVAL_SETTING, (ignored) -> {});
199204
return null;
200205
}
201206
}

0 commit comments

Comments
 (0)