zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nkalmar <...@git.apache.org>
Subject [GitHub] zookeeper issue #590: [ZOOKEEPER-1177] Add the memory optimized watch manage...
Date Thu, 06 Sep 2018 10:34:02 GMT
Github user nkalmar commented on the issue:

    https://github.com/apache/zookeeper/pull/590
  
    systest will go to src/test/java , from my side, you can put the bench in org.apache.zookeeper.test.system
. Thinking about it, that's a pretty good place.
    
    No need to create main directory or anything, I will move all the files anyway. Just move
the files amongst the others in systest. (Hopefully no package level dependency, I didn't
check)
    
    But there's a chance you will have to rebase if this this PR cannot be merged before the
movement of all the remaining files as the directory refactor's last step. Sorry about that
in advance... I'm not going to be the most popular with that PR :(


---

Mime
View raw message