Vladsz83 commented on code in PR #11957:
URL: https://github.com/apache/ignite/pull/11957#discussion_r2017317034


##########
modules/calcite/src/test/java/org/apache/ignite/internal/processors/query/calcite/planner/JoinCommutePlannerTest.java:
##########
@@ -76,6 +82,100 @@ public class JoinCommutePlannerTest extends 
AbstractPlannerTest {
         );
     }
 
+    /** Ensures that join commute rules are disabled if joins count is too 
high. Checks the tables order in the query. */
+    @Test
+    public void testCommuteDisabledForManyJoins() throws Exception {

Review Comment:
   Probably. I'll fix it in the next ticket. We can use disable_rule or change 
join types. Looks simpl.



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to