[ https://issues.apache.org/jira/browse/ZOOKEEPER-775?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12874119#action_12874119 ]
Patrick Hunt commented on ZOOKEEPER-775: ---------------------------------------- My initial feedback: The build fails with proto buff errors. I already have protobuf installed for other reasons, can't mvn pull down the protobuf version it needs vs relying on the system wide libs? Including the bk/zk jar files in server/lib is a really bad practice. Why do we have to do this vs using versions built from the current tree? If we have to do ths we should also indicate that this is not a release version - typically in mvn this means putting something like "SNAPSHOT" into the name of the jars. The README should have a tiny bit of text describing, at a very high level, what hedwig is. The contents of the scripts directory are not executable by default, we should either fix this in the patch or note it for committer to fix at commit time. ZooKeeperTestBase.java has windows eol characters. Perhaps run dos2unix on all the files and refresh the patch? (looks like all the tests suffer from this?) RAT identified the following files which seem like they should have license headers added: {quote} ./formatter.xml ./pom.xml ./client/pom.xml ./client/src/main/cpp/Makefile ./client/src/main/cpp/log4cpp.conf ./client/src/main/resources/log4j.properties ./doc/build.txt ./doc/dev.txt ./doc/doc.txt ./doc/user.txt ./protocol/Makefile ./protocol/pom.xml ./protocol/src/main/protobuf/PubSubProtocol.proto ./scripts/analyze.py ./scripts/hw.bash ./scripts/quote ./server/pom.xml {quote} > A large scale pub/sub system > ---------------------------- > > Key: ZOOKEEPER-775 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-775 > Project: Zookeeper > Issue Type: New Feature > Components: contrib > Reporter: Benjamin Reed > Assignee: Benjamin Reed > Fix For: 3.4.0 > > Attachments: libs.zip, ZOOKEEPER-775.patch, ZOOKEEPER-775_2.patch > > > we have developed a large scale pub/sub system based on ZooKeeper and > BookKeeper. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.