[ 
https://issues.apache.org/jira/browse/BOOKKEEPER-705?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14579535#comment-14579535
 ] 

Flavio Junqueira commented on BOOKKEEPER-705:
---------------------------------------------

[~hustlmsp], looks good, I just have a couple of comments:

# Do you really need expireZooKeeperSession? Can't you just close the zk handle 
directly?
# "// start four new bookies" should be three instead.
# "...due to bookie watcher could not survive..." -> "..due to bookie watcher 
not surviving..."

> handle session expire event in client
> -------------------------------------
>
>                 Key: BOOKKEEPER-705
>                 URL: https://issues.apache.org/jira/browse/BOOKKEEPER-705
>             Project: Bookkeeper
>          Issue Type: Sub-task
>          Components: bookkeeper-client, bookkeeper-server
>            Reporter: Sijie Guo
>            Assignee: Sijie Guo
>             Fix For: 4.4.0
>
>         Attachments: BOOKKEEPER-705.diff, BOOKKEEPER-705.v2.patch
>
>
> handling zookeeper session expire event in bookie server & bookie watcher.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to