Github user anmolnar commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/590#discussion_r214866254
--- Diff: build.xml ---
@@ -119,6 +119,7 @@ xmlns:cs="antlib:com.puppycrawl.tools.checkstyle.ant">
<property name="test.java.classes"
value="${test.java.build.dir}/classes"/>
<property name="test.src.dir" value="${src.dir}/java/test"/>
<property name="systest.src.dir" value="${src.dir}/java/systest"/>
+ <property name="bench.src.dir" value="${src.dir}/java/bench"/>
--- End diff --
I think this new dir should be added to classpath of `eclipse` task too.
---