CR Fixes

GalLalouche · GalLalouche · commit 26af770e9170 · 2025-07-06T14:57:40.000+03:00
diff --git a/server/src/main/java/org/elasticsearch/tasks/TaskManager.java b/server/src/main/java/org/elasticsearch/tasks/TaskManager.java
@@ -97,10 +97,13 @@ public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHea
         this(settings, threadPool, taskHeaders, Tracer.NOOP);
     }
 
+    // For testing
     public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHeaders, Tracer tracer) {
         this(settings, threadPool, taskHeaders, tracer, UUIDs.randomBase64UUID());
     }
 
+    // TODO Both of the above overloads should be moved to the test package.
+
     public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHeaders, Tracer tracer, String nodeId) {
         this.threadPool = threadPool;
         this.taskHeaders = Set.copyOf(taskHeaders);

Original file line number	Diff line number	Diff line change
`@@ -97,10 +97,13 @@ public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHea`
`97`	`97`	`this(settings, threadPool, taskHeaders, Tracer.NOOP);`
`98`	`98`	`}`
`99`	`99`
	`100`	`+ // For testing`
`100`	`101`	`public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHeaders, Tracer tracer) {`
`101`	`102`	`this(settings, threadPool, taskHeaders, tracer, UUIDs.randomBase64UUID());`
`102`	`103`	`}`
`103`	`104`
	`105`	`+ // TODO Both of the above overloads should be moved to the test package.`
	`106`	`+`
`104`	`107`	`public TaskManager(Settings settings, ThreadPool threadPool, Set<String> taskHeaders, Tracer tracer, String nodeId) {`
`105`	`108`	`this.threadPool = threadPool;`
`106`	`109`	`this.taskHeaders = Set.copyOf(taskHeaders);`