accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Yu (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (ACCUMULO-2898) Lack of synchronization for childrenCache in ZooCache#getChildren()
Date Mon, 28 Sep 2015 04:01:04 GMT

     [ https://issues.apache.org/jira/browse/ACCUMULO-2898?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ted Yu updated ACCUMULO-2898:
-----------------------------
    Description: 
For run() method:
{code}
      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);
{code}

access to childrenCache isn't protected by ZooCache.this

  was:
For run() method:
{code}
      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);
{code}
access to childrenCache isn't protected by ZooCache.this


> Lack of synchronization for childrenCache in ZooCache#getChildren()
> -------------------------------------------------------------------
>
>                 Key: ACCUMULO-2898
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2898
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Priority: Minor
>             Fix For: 1.8.0
>
>
> For run() method:
> {code}
>       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);
> {code}
> access to childrenCache isn't protected by ZooCache.this



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message