[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-775?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12899797#action_12899797
 ] 

Ivan Kelly commented on ZOOKEEPER-775:
--------------------------------------

1. You can't convey executableness in a patch afaik, but when this is to be 
submitted the submitter shall do it.

2. I think just specifying that protobuf is needed should be enough. Many linux 
distros already package it. 

3. We're currently looking into the tr1 issue. We've hit across this issue with 
some older distros, but this should all be available from gcc 4.0 onwards, 
which is 3 years old. The optional use of boost is an option we will be looking 
into though.

4. I prefer the all lower case for c++ code, as this is generally the approach 
most open source projects take. For example, everything in /usr/include/c++ is 
lowercase. 

> 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, libs_2.zip, ZOOKEEPER-775.patch, 
> ZOOKEEPER-775.patch, ZOOKEEPER-775.patch, ZOOKEEPER-775_2.patch, 
> ZOOKEEPER-775_3.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.

Reply via email to