deniskuzZ commented on a change in pull request #2272:
URL: https://github.com/apache/hive/pull/2272#discussion_r632445375



##########
File path: 
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java
##########
@@ -466,25 +466,30 @@ public Configuration getConf() {
   @Override
   @RetrySemantics.ReadOnly
   public GetOpenTxnsInfoResponse getOpenTxnsInfo() throws MetaException {
-    return getOpenTxnsList(true).toOpenTxnsInfoResponse();
+    return getOpenTxnsList(true, null).toOpenTxnsInfoResponse();

Review comment:
       Please create a variation of getOpenTxnsList with just 1 argument that 
requires a new db connection. 
   ```
   private OpenTxnList getOpenTxnsList(boolean infoFields){
   return getOpenTxnsList(infoFields, 
getDbConn(Connection.TRANSACTION_READ_COMMITTED))
   }
   ```




-- 
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



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

Reply via email to