zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Patrick Hunt (JIRA)" <j...@apache.org>
Subject [jira] Commented: (ZOOKEEPER-63) Race condition in client close() operation
Date Wed, 23 Jul 2008 18:49:31 GMT

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

Patrick Hunt commented on ZOOKEEPER-63:
---------------------------------------

Correct me if I'm wrong but I don't believe this "fixes" anything, it merely ignores the problem.


1) there is still the stated race condition
2) what is causing the underlying hang? this should not be happening and needs to be addressed
directly, may even be related to item 1

3) if the timeout is reached a warning should be issued in the log

My current vote is -1 


> Race condition in client close() operation
> ------------------------------------------
>
>                 Key: ZOOKEEPER-63
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-63
>             Project: Zookeeper
>          Issue Type: Bug
>          Components: java client
>            Reporter: Patrick Hunt
>            Assignee: Benjamin Reed
>         Attachments: patch_ZOOKEEPER-63.patch
>
>
> There is a race condition in the java close operation on ZooKeeper.java.
> Client is sending a disconnect request to the server. Server will close any open connections
with the client when it receives this. If the client has not yet shutdown it's subthreads
(event/send threads for example) these threads may consider the condition an error. We see
this alot in the tests where the clients output error logs because they are unaware that a
disconnection has been requested by the client.
> Ben mentioned: perhaps we just have to change state to closed (on client) before sending
disconnect request.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message