You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@zookeeper.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2019/08/09 19:53:00 UTC

[jira] [Commented] (ZOOKEEPER-3347) Improve PathTrie Consistency

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

Hudson commented on ZOOKEEPER-3347:
-----------------------------------

ABORTED: Integrated in Jenkins build Zookeeper-trunk-single-thread #490 (See [https://builds.apache.org/job/Zookeeper-trunk-single-thread/490/])
ZOOKEEPER-3347: Improve PathTrie Consistency (andor: rev 04cc5cae1e7d4b007490e68109a676b88ce790a3)
* (edit) zookeeper-server/src/test/java/org/apache/zookeeper/server/DataTreeTest.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/server/DataTree.java
* (add) zookeeper-server/src/test/java/org/apache/zookeeper/common/PathTrieTest.java
* (edit) zookeeper-server/src/main/java/org/apache/zookeeper/common/PathTrie.java


> Improve PathTrie Consistency 
> -----------------------------
>
>                 Key: ZOOKEEPER-3347
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3347
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: server
>            Reporter: David Mollitor
>            Assignee: David Mollitor
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.6.0
>
>          Time Spent: 3h 50m
>  Remaining Estimate: 0h
>
> There is a bunch of synchronization that occurs in the {{PathTrie}}.  Each node in the tree requires a lock to view its children, so to traverse a tree that is 8 nodes deep, it is required to lock 8 different times.  Also, I'm not really sure that the locking is consistent; a node deep in the tree can be negatively impacted by another thread deleting the node's parent at the same time.



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)