pan3793 commented on code in PR #7113: URL: https://github.com/apache/kyuubi/pull/7113#discussion_r2168161638
########## docs/deployment/engine_on_yarn.md: ########## @@ -219,7 +219,11 @@ $ echo "export HADOOP_CONF_DIR=/path/to/hadoop/conf" >> $KYUUBI_HOME/conf/kyuubi #### Required Environment Variable -The `FLINK_HADOOP_CLASSPATH` is required, too. +The `FLINK_HADOOP_CLASSPATH` is required unless the necessary Hadoop client jars (such as `hadoop-client` or +`flink-shaded-hadoop`) have already been placed in the Flink lib directory (`$FLINK_HOME/lib`). + +If the jars are not present in `$FLINK_HOME/lib`, you must set `FLINK_HADOOP_CLASSPATH` to include the appropriate +Hadoop client jars. Review Comment: this line looks redundant, let's keep the words simple. ```suggestion ``` -- 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. To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org For additional commands, e-mail: notifications-h...@kyuubi.apache.org
