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

ASF GitHub Bot commented on DRILL-3171:
---------------------------------------

Github user sudheeshkatkam commented on a diff in the pull request:

    https://github.com/apache/drill/pull/260#discussion_r45006882
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/zk/ZkAbstractStore.java
 ---
    @@ -23,30 +23,35 @@
     import java.util.Iterator;
     import java.util.List;
     import java.util.Map.Entry;
    +import java.util.concurrent.TimeUnit;
     
     import org.apache.curator.framework.CuratorFramework;
    +import org.apache.curator.framework.api.ACLBackgroundPathAndBytesable;
     import org.apache.curator.framework.recipes.cache.ChildData;
     import org.apache.curator.framework.recipes.cache.PathChildrenCache;
     import 
org.apache.curator.framework.recipes.cache.PathChildrenCache.StartMode;
    -import org.apache.drill.exec.rpc.data.DataTunnel;
    +import org.apache.curator.framework.recipes.locks.InterProcessLock;
     import org.apache.drill.exec.store.sys.PStoreConfig;
     import org.apache.zookeeper.CreateMode;
     
     import com.google.common.base.Preconditions;
     import com.google.common.collect.Lists;
    +import org.apache.zookeeper.KeeperException;
     
     /**
      * This is the abstract class that is shared by ZkPStore (Persistent 
store) and ZkEStore (Ephemeral Store)
      * @param <V>
      */
     public abstract class ZkAbstractStore<V> implements AutoCloseable {
    -  static final org.slf4j.Logger logger = 
org.slf4j.LoggerFactory.getLogger(ZkAbstractStore.class);
    +  private static final org.slf4j.Logger logger = 
org.slf4j.LoggerFactory.getLogger(ZkAbstractStore.class);
     
    -  protected CuratorFramework framework;
    -  protected PStoreConfig<V> config;
    +  protected static final long LOCK_TIMEOUT_MILLIS = 3000;
    --- End diff --
    
    unused.


> Storage Plugins : Two processes tried to update the storage plugin at the 
> same time
> -----------------------------------------------------------------------------------
>
>                 Key: DRILL-3171
>                 URL: https://issues.apache.org/jira/browse/DRILL-3171
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - Information Schema
>    Affects Versions: 1.0.0
>            Reporter: Rahul Challapalli
>            Assignee: Deneche A. Hakim
>              Labels: test
>             Fix For: Future
>
>
> Commit Id# : bd8ac4fca03ad5043bca27fbc7e0dec5a35ac474
> We have seen this issue happen with the below steps
>    1. Clear out the zookeeper
>    2. Update the storage plugin using the rest API on one of the node
>    3. Submit 10 queries concurrently
> With randomized foreman node selection, the node executing the query might 
> not have the updated storage plugins info. This could be causing the issue.
> - Rahul



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

Reply via email to