wangyang0918 commented on a change in pull request #10037: [FLINK-14561] Don't write FLINK_PLUGINS_DIR env variable to Configuration URL: https://github.com/apache/flink/pull/10037#discussion_r340410753
########## File path: flink-core/src/main/java/org/apache/flink/configuration/ConfigConstants.java ########## @@ -2012,6 +2012,9 @@ /** The environment variable name which contains the location of the plugins folder. */ public static final String ENV_FLINK_PLUGINS_DIR = "FLINK_PLUGINS_DIR"; + /** The default Flink plugins directory is none has been specified via {@link #ENV_FLINK_PLUGINS_DIR}. */ Review comment: Is it a typo? is -> if /** The default Flink plugins directory if none has been specified via {@link #ENV_FLINK_PLUGINS_DIR}. */ ---------------------------------------------------------------- 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 With regards, Apache Git Services