jcamachor commented on a change in pull request #794: HIVE-21884
URL: https://github.com/apache/hive/pull/794#discussion_r341449442
 
 

 ##########
 File path: ql/src/java/org/apache/hadoop/hive/ql/lockmgr/DbTxnManager.java
 ##########
 @@ -420,6 +429,28 @@ LockState acquireLocks(QueryPlan plan, Context ctx, 
String username, boolean isB
     return lockState;
   }
 
+  private Collection<LockComponent> getGlobalLocks(Configuration conf) {
+    String lockNames = conf.get(ConfVars.HIVE_QUERY_EXCLUSIVE_LOCK.varname);
+    if (StringUtils.isEmpty(lockNames)) {
+      return Collections.emptyList();
+    }
+    List<LockComponent> globalLocks = new ArrayList<LockComponent>();
+    for (String lockName : lockNames.split(",")) {
 
 Review comment:
   Can we end up having multiple locks here for schedule query? That should not 
be the case?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to