[GitHub] [flink] becketqin commented on a diff in pull request #22509: [FLINK-31983] Add yarn Acls capability to Flink containers

2023-06-14 Thread via GitHub
becketqin commented on code in PR #22509: URL: https://github.com/apache/flink/pull/22509#discussion_r1230258326 ## flink-yarn/src/main/java/org/apache/flink/yarn/configuration/YarnConfigOptions.java: ## @@ -231,6 +231,27 @@ public class YarnConfigOptions {

[GitHub] [flink] becketqin commented on a diff in pull request #22509: [FLINK-31983] Add yarn Acls capability to Flink containers

2023-06-09 Thread via GitHub
becketqin commented on code in PR #22509: URL: https://github.com/apache/flink/pull/22509#discussion_r1223687864 ## flink-yarn/src/main/java/org/apache/flink/yarn/Utils.java: ## @@ -620,4 +626,54 @@ public static YarnConfiguration getYarnConfiguration( return

[GitHub] [flink] becketqin commented on a diff in pull request #22509: [FLINK-31983] Add yarn Acls capability to Flink containers

2023-06-08 Thread via GitHub
becketqin commented on code in PR #22509: URL: https://github.com/apache/flink/pull/22509#discussion_r1223048306 ## flink-yarn/src/main/java/org/apache/flink/yarn/Utils.java: ## @@ -620,4 +626,38 @@ public static YarnConfiguration getYarnConfiguration( return

[GitHub] [flink] becketqin commented on a diff in pull request #22509: [FLINK-31983] Add yarn Acls capability to Flink containers

2023-05-09 Thread via GitHub
becketqin commented on code in PR #22509: URL: https://github.com/apache/flink/pull/22509#discussion_r1189255125 ## docs/layouts/shortcodes/generated/yarn_config_configuration.html: ## @@ -182,5 +182,15 @@ String Specify YARN node label for the Flink