[GitHub] [hive] belugabehr commented on a change in pull request #2299: HIVE-25141: Review Error Level Logging in HMS Module

[GitBox <gi...@apache.org>]

belugabehr commented on a change in pull request #2299:
URL: https://github.com/apache/hive/pull/2299#discussion_r639913863



##########
File path: ql/src/test/org/apache/hadoop/hive/metastore/txn/TestTxnHandler.java
##########
@@ -1692,7 +1692,7 @@ public void testReplOpenTxn() throws Exception {
     int numTxn = 50000;
     String[] output = TestTxnDbUtil.queryToString(conf, "SELECT MAX(\"TXN_ID\") + 1 FROM \"TXNS\"").split("\n");
     long startTxnId = Long.parseLong(output[1].trim());
-    txnHandler.setOpenTxnTimeOutMillis(30000);
+    txnHandler.setOpenTxnTimeOutMillis(60000);

Review comment:
       This timeout (failing test) was giving me a lot of heartburn.  I have been playing around with it on `master` branch and it's pretty sensitive to change.  Lowering it just a little causes timeout failures.  I have bumped it here to get it to pass and to make this test less flaky.




-- 
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:
users@infra.apache.org



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