[GitHub] zookeeper issue #74: ServerConfig#parse(String[]) parse params has problems

2016-12-31 Thread eribeiro
Github user eribeiro commented on the issue: https://github.com/apache/zookeeper/pull/74 @hanm I didn't tested on a git repo. The guy who told me about this trick mentioned the Spark project guys use it (even though they don't use it on the Kafka project). --- If your

[GitHub] zookeeper issue #74: ServerConfig#parse(String[]) parse params has problems

2016-12-30 Thread hanm
Github user hanm commented on the issue: https://github.com/apache/zookeeper/pull/74 @eribeiro You mentioned in a dev mailing thread that a PR can be closed by a "dummy commit" - to quote what you said: > The 'dummy commit' is a Github trick. In any commit message you can write

[GitHub] zookeeper issue #74: ServerConfig#parse(String[]) parse params has problems

2016-12-29 Thread eribeiro
Github user eribeiro commented on the issue: https://github.com/apache/zookeeper/pull/74 This issue was solved by https://github.com/apache/zookeeper/pull/125 so this PR can be closed PS: @hanm and @fpj and @phunt, afaik, you have to ask INFRA to close it. --- If your