[ 
https://issues.apache.org/jira/browse/FLINK-6498?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16053838#comment-16053838
 ] 

ASF GitHub Bot commented on FLINK-6498:
---------------------------------------

Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4123#discussion_r122684018
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/util/ZooKeeperUtils.java 
---
    @@ -391,11 +370,11 @@ public static String generateZookeeperPath(String 
root, String namespace) {
                 * Return the configured {@link ZkClientACLMode}.
                 *
                 * @param config The config to parse
    -            * @return Configured ACL mode or {@link 
ConfigConstants#DEFAULT_HA_ZOOKEEPER_CLIENT_ACL} if not
    +            * @return Configured ACL mode or {@link 
HighAvailabilityOptions#ZOOKEEPER_CLIENT_ACL} if not
                 * configured.
                 */
                public static ZkClientACLMode fromConfig(Configuration config) {
    -                   String aclMode = 
config.getString(ConfigConstants.HA_ZOOKEEPER_CLIENT_ACL, null);
    +                   String aclMode = 
config.getString(HighAvailabilityOptions.ZOOKEEPER_CLIENT_ACL, null);
    --- End diff --
    
    `null` argument can be removed.


> Migrate Zookeeper configuration options
> ---------------------------------------
>
>                 Key: FLINK-6498
>                 URL: https://issues.apache.org/jira/browse/FLINK-6498
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Distributed Coordination
>            Reporter: Chesnay Schepler
>            Assignee: Fang Yong
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to