korlov42 commented on a change in pull request #8590:
URL: https://github.com/apache/ignite/pull/8590#discussion_r548841625
##########
File path:
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/CalciteBasicSecondaryIndexIntegrationTest.java
##########
@@ -120,10 +121,11 @@ public void testIndexLoopJoin() {
.check();
}
+ /** */
@Test
public void testMergeJoin() {
assertQuery("" +
- "SELECT d1.name, d2.name FROM Developer d1, Developer d2 WHERE
d1.depId = d2.depId")
+ "SELECT /*+ DISABLE_RULE('CorrelatedNestedLoopJoin') */ d1.name,
d2.name FROM Developer d1, Developer d2 WHERE d1.depId = d2.depId")
Review comment:
because in the middle of the task progress this test fails several times
because of changed plan. Since this test should verify only correctness of
MergeJoin execution, I think it's OK to disable other algorithms explicitly
----------------------------------------------------------------
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:
[email protected]
