[GitHub] zookeeper issue #371: ZOOKEEPER-2814: Ignore space after comma in connection...

2017-10-13 Thread dolphy17 
Github user dolphy17 commented on the issue:

https://github.com/apache/zookeeper/pull/371
  
I prefer the regex solution too as @mfenes did.


---

[GitHub] zookeeper issue #371: ZOOKEEPER-2814: Ignore space after comma in connection...

2017-10-12 Thread mfenes 
Github user mfenes commented on the issue:

https://github.com/apache/zookeeper/pull/371
  
Regex implementation would be shorter to ignore the spaces and empty 
strings as
```
connectString.replaceAll("\\s", "").replaceAll(",+", ",").split(",") 
```
does the same thing as the new StringUtils.split(String value, String 
separator) implementation in this PR.


---