You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "Chunwei Lei (Jira)" <ji...@apache.org> on 2020/10/29 07:52:00 UTC

[jira] [Comment Edited] (CALCITE-4360) Apply SubstitutionRule first in top-down driven rule apply

    [ https://issues.apache.org/jira/browse/CALCITE-4360?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17222756#comment-17222756 ] 

Chunwei Lei edited comment on CALCITE-4360 at 10/29/20, 7:51 AM:
-----------------------------------------------------------------

BTW, I found all the test cases in TopDownOptTest failed if applying the substitution rule last in the investigation and It's supposed not to be. I would continue to investigate it and file another issue if there is something wrong.


was (Author: chunwei lei):
BTW, I found all the test cases in TopDownOptTest failed if applying the substitution rule last in the investigation and It's supposed not to be. I would continue to investigate it.

> Apply SubstitutionRule first in top-down driven rule apply
> ----------------------------------------------------------
>
>                 Key: CALCITE-4360
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4360
>             Project: Calcite
>          Issue Type: Improvement
>            Reporter: Chunwei Lei
>            Assignee: Chunwei Lei
>            Priority: Major
>              Labels: pull-request-available
>         Attachments: image-2020-10-27-21-55-55-155.png
>
>          Time Spent: 1h 40m
>  Remaining Estimate: 0h
>
> In the current implementation, TopDownRuleQueue adds substitute rules in the end wrongly. The SubstitutionRule should be executed first.
> !image-2020-10-27-21-55-55-155.png!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)