[GitHub] [flink] yuruguo commented on a change in pull request #14477: [FLINK-16005][yarn] Support yarn and hadoop config override

[GitBox]

yuruguo commented on a change in pull request #14477:
URL: https://github.com/apache/flink/pull/14477#discussion_r553255576



##########
File path: docs/_includes/generated/yarn_config_configuration.html
##########
@@ -14,6 +14,20 @@
             <td>String</td>
             <td>If configured, Flink will add this key to the resource profile 
of container request to Yarn. The value will be set to the value of 
external-resource.&lt;resource_name&gt;.amount.</td>
         </tr>
+        <tr>
+            <td><h5>flink.hadoop.&lt;hadoop_key&gt;</h5></td>
+            <td style="word-wrap: break-word;">(none)</td>
+            <td>String</td>
+            <td>A general option to probe Hadoop configuration through prefix 
`flink.hadoop.`. Flink will remove the prefix to get hadoop_key (from <a 
href="https://hadoop.apache.org/docs/stable/hadoop-project-dist/hadoop-common/core-default.xml";>core-default.xml</a>
 and <a 
href="https://hadoop.apache.org/docs/stable/hadoop-project-dist/hadoop-hdfs/hdfs-default.xml";>hdfs-default.xml</a>)
 then set the hadoop_key and value to Hadoop configuration. For example, 
flink.hadoop.dfs.replication=5 in Flink configuration and convert to 
dfs.replication=5 in Hadoop configuration.
+            </td>
+        </tr>
+        <tr>
+            <td><h5>flink.yarn.&lt;key&gt;</h5></td>
+            <td style="word-wrap: break-word;">(none)</td>
+            <td>String</td>
+            <td>A general option to probe Yarn configuration through prefix 
`flink.yarn.`. Flink will remove the prefix `flink.` to get yarn_key (from <a 
href="https://hadoop.apache.org/docs/stable/hadoop-yarn/hadoop-yarn-common/yarn-default.xml";>yarn-default.xml</a>)
 then set the yarn_key and value to Yarn configuration. For example, 
flink.yarn.resourcemanager.container.liveness-monitor.interval-ms=300000 in 
Flink configuration and convert to 
yarn.resourcemanager.container.liveness-monitor.interval-ms=300000 in Yarn 
configuration.
+            </td>
+        </tr>

Review comment:
       Thanks for your guidance, I understand its generation mechanism and I 
will add it.




----------------------------------------------------------------
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