[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813620#comment-17813620 ] Ulrich Kramer commented on CALCITE-6236: [~rubenql] , thanks for the informatio

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813610#comment-17813610 ] Ruben Q L commented on CALCITE-6236: {quote}I am afraid I am just adding negative k

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Stamatis Zampetakis (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813606#comment-17813606 ] Stamatis Zampetakis commented on CALCITE-6236: -- For the record, pre-storin

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813605#comment-17813605 ] Ulrich Kramer commented on CALCITE-6236: Perhaps, it's really better to store t

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813598#comment-17813598 ] Ulrich Kramer commented on CALCITE-6236: Yes, it breaks also my PR. > Enumerab

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813593#comment-17813593 ] Ruben Q L commented on CALCITE-6236: Thanks [~kramerul] . I also started to look in

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-02 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813585#comment-17813585 ] Ulrich Kramer commented on CALCITE-6236: I created a PR based on the filter sel

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813311#comment-17813311 ] Ulrich Kramer commented on CALCITE-6236: My example is quite simple. But I have

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Alessandro Solimando (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813303#comment-17813303 ] Alessandro Solimando commented on CALCITE-6236: --- Equivalent in terms of c

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813280#comment-17813280 ] Ulrich Kramer commented on CALCITE-6236: I would also prefer the solution with

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813279#comment-17813279 ] Ulrich Kramer commented on CALCITE-6236: As far as I understand it, the suggest

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813276#comment-17813276 ] Ruben Q L commented on CALCITE-6236: {quote}Rules create semantically equivalent pl

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Stamatis Zampetakis (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813266#comment-17813266 ] Stamatis Zampetakis commented on CALCITE-6236: -- Rules create semantically

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813259#comment-17813259 ] Ruben Q L commented on CALCITE-6236: [~zabetak] I agree with you that my solution i

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Stamatis Zampetakis (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813258#comment-17813258 ] Stamatis Zampetakis commented on CALCITE-6236: -- If the Filter is the one w

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813247#comment-17813247 ] Ulrich Kramer commented on CALCITE-6236: Another possibility would be to store

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813245#comment-17813245 ] Ulrich Kramer commented on CALCITE-6236: Falling back to {{Join.estimateRowCoun

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Stamatis Zampetakis (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813240#comment-17813240 ] Stamatis Zampetakis commented on CALCITE-6236: -- It's pretty strange to der

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-02-01 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813161#comment-17813161 ] Ulrich Kramer commented on CALCITE-6236: I tested PR 3661 in our environment. I

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17813084#comment-17813084 ] Ulrich Kramer commented on CALCITE-6236: I closed 3660 > EnumerableBatchNested

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Mihai Budiu (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812807#comment-17812807 ] Mihai Budiu commented on CALCITE-6236: -- Should 3660 be closed? > EnumerableBatchN

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812799#comment-17812799 ] Ruben Q L commented on CALCITE-6236: See https://github.com/apache/calcite/pull/366

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812747#comment-17812747 ] Ruben Q L commented on CALCITE-6236: Working on it... > EnumerableBatchNestedLoopJ

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Ulrich Kramer (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812741#comment-17812741 ] Ulrich Kramer commented on CALCITE-6236: It would be great, if you could prepar

[jira] [Commented] (CALCITE-6236) EnumerableBatchNestedLoopJoin uses wrong row count for cost calculation

2024-01-31 Thread Ruben Q L (Jira)
[ https://issues.apache.org/jira/browse/CALCITE-6236?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812730#comment-17812730 ] Ruben Q L commented on CALCITE-6236: [~kramerul], I have taken a quick look at the