You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zookeeper.apache.org by "Steven Cheng (JIRA)" <ji...@apache.org> on 2009/11/13 03:20:39 UTC

[jira] Commented: (ZOOKEEPER-414) zkServer.sh is duplicated in a couple of places

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-414?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12777336#action_12777336 ] 

Steven Cheng commented on ZOOKEEPER-414:
----------------------------------------

Current list of zkServer.sh files:

./src/contrib/zkpython/src/test/zkServer.sh
./src/contrib/rest/src/test/zkServer.sh
./src/c/tests/zkServer.sh
./src/recipes/queue/src/c/tests/zkServer.sh
./src/recipes/lock/src/c/tests/zkServer.sh
./bin/zkServer.sh

All of the extra ones are for testing.  Looked into this a bit, main difference between bin/zkServer.sh and the testing scripts is that the testing scripts delete the zookeeper data.  I'm guessing that it wouldn't be helpful to have "zkServer.sh delete"?

One option is to create bin/testzkServer.sh that handles the deleting and calls zkServer.sh as needed.  Then a conf file just needs to be created and the test suites can be hooked in to the new script.


> zkServer.sh is duplicated in a couple of places
> -----------------------------------------------
>
>                 Key: ZOOKEEPER-414
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-414
>             Project: Zookeeper
>          Issue Type: Improvement
>            Reporter: Benjamin Reed
>            Assignee: Mahadev konar
>             Fix For: 3.3.0
>
>
> we have zkServer.sh in 4 different places. some differ only in whitespace. it would be much more maintainable if there were only one version that everything used.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.