You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/08/15 10:03:13 UTC

[GitHub] [flink] tillrohrmann commented on a change in pull request #9336: [FLINK-13548][Deployment/YARN]Support priority of the Flink YARN application

tillrohrmann commented on a change in pull request #9336: [FLINK-13548][Deployment/YARN]Support priority of the Flink YARN application
URL: https://github.com/apache/flink/pull/9336#discussion_r314246453
 
 

 ##########
 File path: flink-yarn/src/main/java/org/apache/flink/yarn/configuration/YarnConfigOptions.java
 ##########
 @@ -180,6 +180,25 @@
 			" Flink on YARN on an environment with a restrictive firewall, this option allows specifying a range of" +
 			" allowed ports.");
 
+	/**
+	 * A non-negative integer indicating the priority for submitting a Flink YARN application. It will only take effect
+	 * if the Hadoop version >= 2.8.5 and YARN priority scheduling setting is enabled. Larger integer corresponds with
+	 * higher priority. If priority is negative or set to '-1'(default), Flink will unset yarn priority setting and use
+	 * cluster default priority.
+	 *
+	 * @see <a href="https://hadoop.apache.org/docs/r2.8.5/hadoop-yarn/hadoop-yarn-site/CapacityScheduler.html">YARN Capacity Scheduling Doc</a>
+	 */
+	public static final ConfigOption<Integer> APPLICATION_PRIORITY =
+		key("yarn.application.priority")
+			.defaultValue(-1)
+			.withDescription(Description.builder()
+				.text("A non-negative integer indicating the priority for submitting a Flink YARN application. It" +
+					" will only take effect if the Hadoop version >= 2.8.5 and YARN priority scheduling setting is enabled." +
 
 Review comment:
   Why does it only work with Hadoop version >= `2.8.5`? The `ApplicationSubmissionContext` has the `#setPriority` method since version `2.0.4-alpha`? Has the method never been used before `2.8.5`?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services