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

Reply via email to