PakhomovAlexander commented on code in PR #5209:
URL: https://github.com/apache/ignite-3/pull/5209#discussion_r1987359473
##########
modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItDmlTest.java:
##########
@@ -373,9 +373,12 @@ public void testMergeBatch() {
assertQuery("SELECT count(*) FROM test2 WHERE b =
0").returns(10_000L).check();
- sql("MERGE INTO test2 dst USING test1 src ON dst.a = src.a"
+ var longerTimeoutOptions = new
TransactionOptions().readOnly(false).timeoutMillis(TimeUnit.MINUTES.toMillis(2));
Review Comment:
Agreed. But Im afraid a little of this change. The major question is: "Do we
have good enough defaults if we change them for all tests"? Maybe we should
adjust defaults? Make them longer? WDYT?
--
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]
