Mahadev konar updated ZOOKEEPER-121:

    Attachment: ZOOKEEPER-121_3.patch

this patch should fix the issue. We were closing zkdb in syncrequestprocessor 
but since it gets shared across zookeeper server instances, we should only be 
closing it in quorumpeer shutdown. I also added a synchronized in rollLog 
(htough its not necessary but we should have had it).

> SyncRequestProcessor is not closing log stream during shutdown
> --------------------------------------------------------------
>                 Key: ZOOKEEPER-121
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-121
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: server
>            Reporter: Patrick Hunt
>            Assignee: Mahadev konar
>            Priority: Blocker
>             Fix For: 3.3.0
>         Attachments: ZOOKEEPER-121.patch, ZOOKEEPER-121.patch, 
> ZOOKEEPER-121.patch, ZOOKEEPER-121.patch, ZOOKEEPER-121_3.patch, 
> ZOOKEEPER-121_3.patch
> The SyncRequestProcessor is not closing log stream during shutdown. 
> See FIXMEs with this ID in ClientBase.java -- I've commented out the 
> assertion for the time being (checking for logs being deleted), as part of 
> this fix re-enable these asserts and also verify tests on a Windows system.

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