flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-6498) Migrate Zookeeper configuration options
Date Wed, 14 Jun 2017 10:26:00 GMT

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

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_r121910223
  
    --- Diff: flink-core/src/main/java/org/apache/flink/configuration/HighAvailabilityOptions.java
---
    @@ -100,6 +100,45 @@
     			.defaultValue("/flink")
     			.withDeprecatedKeys("recovery.zookeeper.path.root");
     
    +	public static final ConfigOption<String> HA_ZOOKEEPER_NAMESPACE =
    +			key("high-availability.zookeeper.path.namespace")
    +			.noDefaultValue();
    +
    +	public static final ConfigOption<String> HA_ZOOKEEPER_LATCH_PATH =
    +			key("high-availability.zookeeper.path.latch")
    +			.defaultValue("/leaderlatch")
    +			.withDeprecatedKeys("recovery.zookeeper.path.latch");
    +
    +	/** ZooKeeper root path (ZNode) for job graphs. */
    +	public static final ConfigOption<String> HA_ZOOKEEPER_JOBGRAPHS_PATH =
    +			key("high-availability.zookeeper.path.jobgraphs")
    +			.defaultValue("/jobgraphs")
    +			.withDeprecatedKeys("recovery.zookeeper.path.jobgraphs");
    +
    +	public static final ConfigOption<String> HA_ZOOKEEPER_LEADER_PATH =
    +			key("high-availability.zookeeper.path.leader")
    +			.defaultValue("/leader")
    +			.withDeprecatedKeys("recovery.zookeeper.path.leader");
    +
    +	/** ZooKeeper root path (ZNode) for completed checkpoints. */
    +	public static final ConfigOption<String> HA_ZOOKEEPER_CHECKPOINTS_PATH =
    +			key("high-availability.zookeeper.path.checkpoints")
    +			.defaultValue("/checkpoints")
    +			.withDeprecatedKeys("recovery.zookeeper.path.checkpoints");
    +
    +	/** ZooKeeper root path (ZNode) for checkpoint counters. */
    +	public static final ConfigOption<String> HA_ZOOKEEPER_CHECKPOINT_COUNTER_PATH
=
    --- End diff --
    
    well isn't that great.
    
    OK, keep the prefix for the HA options, but remove it for `ZOOKEEPER_CLIENT_ACL`.


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

Mime
View raw message