Github user phunt commented on the issue:

    https://github.com/apache/zookeeper/pull/497
  
    +1. Looks reasonable however:
    
    1) this problem also exists in 
src/java/test/org/apache/zookeeper/test/ReadOnlyModeTest.java - do you want to 
fix it as part of this change or handle as a separate jira? (either way is fine 
with me - separate might make it simpler to review/commit and make progress 
tbh, but up to you)
    
    2) regardless 1) above, this doesn't apply to branch-3.4 - the original 
jira says it affects 3.4, I'm assuming we want to fix there as well? Could you 
create a separate PR for this? I'd like to commit both at the same time.


---

Reply via email to