hadoop-zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Hunt (JIRA)" <j...@apache.org>
Subject [jira] Commented: (ZOOKEEPER-775) A large scale pub/sub system
Date Tue, 01 Jun 2010 17:39:44 GMT

    [ 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.


Mime
View raw message