zratkai commented on code in PR #4690:
URL: https://github.com/apache/hive/pull/4690#discussion_r1343782363
##########
itests/hive-unit/src/test/java/org/apache/hadoop/hive/ql/txn/compactor/TestCrudCompactorOnTez.java:
##########
@@ -146,7 +146,7 @@ private void
testRebalanceCompactionWithParallelDeleteAsSecond(boolean optimisti
}
aborted = true;
Assert.assertEquals(e.getCause().getClass(), LockException.class);
- Assert.assertEquals(e.getCauseMessage(), "Transaction manager has
aborted the transaction txnid:21. Reason: Aborting [txnid:21,24] due to a
write conflict on default/rebalance_test committed by [txnid:20,24] d/u");
+ Assert.assertEquals( "Transaction manager has aborted the transaction
txnid:21. Reason: Aborting [txnid:21,24] due to a write conflict on
default/rebalance_test committed by [txnid:20,24] d/u", e.getCauseMessage());
Review Comment:
I can understand your point, but the build environment is really unstable,
and I do not want to risk a failed build because of this change.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
