[jira] [Commented] (STORM-906) Flux "--local --zookeeper" dose not work

["Jungtaek Lim (JIRA)" <ji...@apache.org>]

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

Jungtaek Lim commented on STORM-906:
------------------------------------

It is not a bug but limitation of current LocalCluster.

Flux uses LocalCluster(String zkHost, Long zkPort) when "local" and "zookeeper" are configured.

{code}
options.addOption(option(1, "z", OPTION_ZOOKEEPER, "host:port", "When running in local mode, use the ZooKeeper at the specified <host>:<port> instead of the in-process ZooKeeper. (requires Storm 0.9.3 or later)"));
{code}

I'm wondering that we may want to support multiple zk servers with local cluster.
If we're fine to leave as is, this issue can be resolved as "Not A Problem".

> Flux "--local --zookeeper" dose not work
> ----------------------------------------
>
>                 Key: STORM-906
>                 URL: https://issues.apache.org/jira/browse/STORM-906
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: Flux
>    Affects Versions: 0.10.0
>            Reporter: Xin Wang
>            Priority: Minor
>
> [description]
> -z,--zookeeper <host:port>
> When running in local mode, use the ZooKeeper at the specified <host>:<port>  instead of the in-process ZooKeeper.
> But when using zookeeper string like 'host-A:2181,host-B:2181,host-C:2181', Flux dose not work well.
> [command]
> {code}
> storm jar topo.jar org.apache.storm.flux.Flux --local --zookeeper host-A:2181,host-B:2181,host-C:2181 topo.yaml
> {code}
> [code]
> Flux.java:
> {code}
> ...
> if(zkStr.contains(":")){
>                         String[] hostPort = zkStr.split(":");
>                         zkHost = hostPort[0];
>                         zkPort = hostPort.length > 1 ? Long.parseLong(hostPort[1]) : DEFAULT_ZK_PORT;
>  }
> ...
> {code}



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