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

Flavio Paiva Junqueira commented on ZOOKEEPER-507:
--------------------------------------------------

One small comment at this point is that we have no lib folder under bookkeeper, 
and Pat tells me that putting under PROJECT_ROOT/src/java/lib is not ok. So, it 
doesn't compile without modifying build.xml. 

I see two options:

# We create a bookkeeper/src/java/lib folder and add the netty jar. In this 
case, we edit build.xml to set its classpath appropriately;
# We make it use ivy and fetch from a repository.

Another point is that for the new bookie classes, I haven't seen javadocs or 
comments at all. I think it would be great to have some in there.



> Improve error handling of BookKeeper client
> -------------------------------------------
>
>                 Key: ZOOKEEPER-507
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-507
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: contrib-bookkeeper
>            Reporter: Flavio Paiva Junqueira
>            Assignee: Utkarsh Srivastava
>         Attachments: ZOOKEEPER-507.patch
>
>
> Error handling is far from ideal currently in the BookKeeper client.

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