[jira] [Updated] (ZOOKEEPER-1785) Small fix in zkServer.sh to support new configuration format

2013-10-09 Thread Patrick Hunt (JIRA)

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

Patrick Hunt updated ZOOKEEPER-1785:


Component/s: scripts

> Small fix in zkServer.sh to support new configuration format
> 
>
> Key: ZOOKEEPER-1785
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1785
> Project: ZooKeeper
>  Issue Type: Bug
>  Components: scripts
>Affects Versions: 3.5.0
>Reporter: Alexander Shraer
>Assignee: Alexander Shraer
>Priority: Minor
> Fix For: 3.5.0
>
> Attachments: zkServersh.patch
>
>
> The problem can be reproduced by running a server with the following type of 
> config file:
> dataDir=/Users/shralex/zookeeper-test/zookeeper1
> syncLimit=2
> initLimit=5
> tickTime=2000
> server.1=localhost:2721:2731:participant;2791
> server.2=localhost:2722:2732:participant;2792
> and then trying to do "zkServer.sh status"
> Here I specified the servers using the new config format but still used the 
> static config file and didn't include the "clientPort" key.
> zkServer.sh already supports the new configuration format, but expects server 
> spec to appear in the dynamic config file if it uses the new format.
> So in the example above it will not find the client port. 
> The current logic for executing something like 'zkServer.sh status'  is:
> 1. Look for clientPort keyword in the static config file
> 2. Look for the client port in the server spec in the dynamic config file
> The attached patch adds an intermediate step:
> 1'. Look for the client port in the server spec in the static config file



--
This message was sent by Atlassian JIRA
(v6.1#6144)


[jira] [Updated] (ZOOKEEPER-1785) Small fix in zkServer.sh to support new configuration format

2013-10-08 Thread Alexander Shraer (JIRA)

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

Alexander Shraer updated ZOOKEEPER-1785:


Attachment: zkServersh.patch

> Small fix in zkServer.sh to support new configuration format
> 
>
> Key: ZOOKEEPER-1785
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-1785
> Project: ZooKeeper
>  Issue Type: Bug
>Affects Versions: 3.5.0
>Reporter: Alexander Shraer
>Assignee: Alexander Shraer
>Priority: Minor
> Fix For: 3.5.0
>
> Attachments: zkServersh.patch
>
>
> The problem can be reproduced by running a server with the following type of 
> config file:
> dataDir=/Users/shralex/zookeeper-test/zookeeper1
> syncLimit=2
> initLimit=5
> tickTime=2000
> server.1=localhost:2721:2731:participant;2791
> server.2=localhost:2722:2732:participant;2792
> and then trying to do "zkServer.sh status"
> Here I specified the servers using the new config format but still used the 
> static config file and didn't include the "clientPort" key.
> zkServer.sh already supports the new configuration format, but expects server 
> spec to appear in the dynamic config file if it uses the new format.
> So in the example above it will not find the client port. 
> The current logic for executing something like 'zkServer.sh status'  is:
> 1. Look for clientPort keyword in the static config file
> 2. Look for the client port in the server spec in the dynamic config file
> The attached patch adds an intermediate step:
> 1'. Look for the client port in the server spec in the static config file



--
This message was sent by Atlassian JIRA
(v6.1#6144)