[ 
https://issues.apache.org/jira/browse/STORM-498?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Michael Noll updated STORM-498:
-------------------------------
    Description: 
Currently the Kafka spout uses a hardcoded ZK connection timeout of 15 seconds 
in {{external/storm-kafka/src/jvm/storm/kafka/DynamicBrokersReader.java}}:

{code}
_curator = CuratorFrameworkFactory.newClient(
    zkStr,
    Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_SESSION_TIMEOUT)),
    15000,
    new RetryNTimes(Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_TIMES)),
        Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_INTERVAL))));
{code}

We should make this setting configurable, similar to the ZK session timeout 
setting.

  was:
Currently the Kafka spout uses a hardcoded ZK connection timeout of 15 seconds 
in {{external/storm-kafka/src/jvm/storm/kafka/DynamicBrokersReader.java}}:

{code}
            _curator = CuratorFrameworkFactory.newClient(
                    zkStr,
                    
Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_SESSION_TIMEOUT)),
                    15000,
                    new 
RetryNTimes(Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_TIMES)),
                            
Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_INTERVAL))));
{code}

We should make this setting configurable, similar to the ZK session timeout 
setting.


> storm-kafka: make ZK connection timeout configurable in Kafka spout
> -------------------------------------------------------------------
>
>                 Key: STORM-498
>                 URL: https://issues.apache.org/jira/browse/STORM-498
>             Project: Apache Storm (Incubating)
>          Issue Type: Improvement
>    Affects Versions: 0.9.3-incubating
>            Reporter: Michael Noll
>            Assignee: Michael Noll
>            Priority: Minor
>
> Currently the Kafka spout uses a hardcoded ZK connection timeout of 15 
> seconds in 
> {{external/storm-kafka/src/jvm/storm/kafka/DynamicBrokersReader.java}}:
> {code}
> _curator = CuratorFrameworkFactory.newClient(
>     zkStr,
>     Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_SESSION_TIMEOUT)),
>     15000,
>     new 
> RetryNTimes(Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_TIMES)),
>         Utils.getInt(conf.get(Config.STORM_ZOOKEEPER_RETRY_INTERVAL))));
> {code}
> We should make this setting configurable, similar to the ZK session timeout 
> setting.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to