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