Flavio Paiva Junqueira updated ZOOKEEPER-436:

    Attachment: ZOOKEEPER-436.patch

I updated Erwin's patch in the following way:

# The patch committed yesterday for ZOOKEEPER-709 generated a conflict, so I 
fixed it;
# One of the tests was failing for me because we were not closing the zookeeper 
client object when shutting down a bookie. In general it shouldn't be a problem 
because the ephemeral znode goes away once the session expires, but in the case 
of testBookieRecovery, we shut down a bookie and re-start it right away, so 
there isn't enough time for expiration. Consequently, I have added a line to 
close the zk client object if zk is not null (it can be null if no server is 
passes as a parameter).

> Bookies should auto register to ZooKeeper
> -----------------------------------------
>                 Key: ZOOKEEPER-436
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-436
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: contrib-bookkeeper
>            Reporter: Benjamin Reed
>             Fix For: 3.3.0
>         Attachments: ZOOKEEPER-436.patch, ZOOKEEPER-436.patch
> currently bookies have to be manually added to ZooKeeper to be used in a 
> BookKeeper service. we should be able to just start up a bookkie, point it at 
> ZooKeeper, and have it get auto integrated in.

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