zhuzhurk commented on a change in pull request #13284:
URL: https://github.com/apache/flink/pull/13284#discussion_r485390221



##########
File path: 
flink-core/src/main/java/org/apache/flink/configuration/JobManagerOptions.java
##########
@@ -339,6 +340,23 @@
                                .list(
                                        text("'ng': new generation scheduler"))
                                .build());
+
+       /**
+        * Config parameter determining the scheduling strategy.
+        */
+       @Documentation.ExcludeFromDocumentation("User normally should not be 
expected to change this config.")
+       public static final ConfigOption<String> SCHEDULING_STRATEGY =

Review comment:
       I tried it regarding this comment but then realized `JobManagerOptions` 
is in flink-core.
   So I think it's better to not add such an enum to flink-core, given that it 
is for a temporary option which will probably be removed in release 1.13.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to