Menu
▾
▴
[Zookeeper-developer] [ zookeeper-Patches-1982992 ] concurrency issue in the zookeeper leader code.
|
From: SourceForge.net <no...@so...> - 2008-06-03 18:05:30
|
Patches item #1982992, was opened at 2008-06-02 21:38 Message generated for change (Comment added) made by breed You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=1008546&aid=1982992&group_id=209147 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: server Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Mahadev Konar (mahadevkonar) Assigned to: Benjamin Reed (breed) Summary: concurrency issue in the zookeeper leader code. Initial Comment: The Datatree.ephemerals has a hashset of ephermal nodes for a given session. This list is not synchronized. This was a part of the bug that one of our users found. One of the sessions was getting expired and some other session was deleting ephemeral nodes from the session getting expired. This caused a concurrency modification excpetion while we called hashlist.clone() and datatree.getephermals() --- The fix is simple to put synchronized blocks around the accesses to the list. ---------------------------------------------------------------------- >Comment By: Benjamin Reed (breed) Date: 2008-06-03 11:05 Message: Logged In: YES user_id=154690 Originator: NO We need synchronized blocks around all accesses. Since the reads are not in synchronized blocks, you will still get concurrent modification exceptions. ---------------------------------------------------------------------- Comment By: Mahadev Konar (mahadevkonar) Date: 2008-06-02 21:39 Message: Logged In: YES user_id=1926680 Originator: YES ben can you review it? ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=1008546&aid=1982992&group_id=209147 |
