[review] Moves DefaultRescaleManagerFactory creation into AdaptiveSch…

…eduler constructor.
XComp · Jun 24, 2024 · 1fca0a1 · 1fca0a1
1 parent d536571
commit 1fca0a1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/...-runtime/src/main/java/org/apache/flink/runtime/scheduler/adaptive/AdaptiveScheduler.java b/...-runtime/src/main/java/org/apache/flink/runtime/scheduler/adaptive/AdaptiveScheduler.java
@@ -283,6 +283,8 @@ public int getMinParallelismChangeForDesiredRescale() {
     }
 
     private final Settings settings;
+    private final RescaleManager.Factory rescaleManagerFactory;
+
     private final JobGraph jobGraph;
 
     private final JobInfo jobInfo;
@@ -360,6 +362,8 @@ public AdaptiveScheduler(
         assertPreconditions(jobGraph);
 
         this.settings = settings;
+        this.rescaleManagerFactory = DefaultRescaleManager.Factory.fromSettings(settings);
+
         this.jobGraph = jobGraph;
         this.jobInfo = new JobInfoImpl(jobGraph.getJobID(), jobGraph.getName());
 
@@ -1044,7 +1048,7 @@ public void goToExecuting(
                         this,
                         userCodeClassLoader,
                         failureCollection,
-                        DefaultRescaleManager.Factory.fromSettings(settings),
+                        rescaleManagerFactory,
                         settings.getMinParallelismChangeForDesiredRescale()));
     }