[ACCUMULO-2898] Lack of synchronization for childrenCache in ZooCache#getChildren()

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.6.5, 1.7.1, 1.8.0
• Assignee: Unassigned
• Reporter: Ted Yu
• Created: 12/Jun/14 17:39
• Updated: 05/Jan/16 20:42
• Resolved: 31/Dec/15 00:54

Description

For run() method:

      public void run(ZooKeeper zooKeeper) throws KeeperException, InterruptedException {

        if (childrenCache.containsKey(zPath))
          return;

        try {
          List<String> children = zooKeeper.getChildren(zPath, watcher);
          childrenCache.put(zPath, children);

access to childrenCache isn't protected by ZooCache.this

Sub-Tasks

• Zoocache utilizes course grained locking causing unnecessary synchronization in many tserver ops: Resolved
• Backport bugfix to 1.6/1.7: Resolved