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

Steve Carlin commented on CALCITE-7243:
---------------------------------------

Heh, was hoping I could get away without it.  I haven't worked much with 
Calcite's unit test framework.

This isn't high priority for me, but figured someone might want it.  I'll try 
to create a unit test soon though.

> Rules in RelDecorrelator not using config of RelBuilder passed in
> -----------------------------------------------------------------
>
>                 Key: CALCITE-7243
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7243
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>            Reporter: Steve Carlin
>            Priority: Major
>
> So I think this is a bug, but I'm not sure I have a complete understanding of 
> the code.
> One rule used in RelDecorrelator is on this line:
> [https://github.com/apache/calcite/blob/main/core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java#L296]
> Here, it uses the rule with what seemingly is the RelBuilder that is passed 
> into the RelDecorrelator as shown in this line:
> [https://github.com/apache/calcite/blob/main/core/src/main/java/org/apache/calcite/sql2rel/RelDecorrelator.java#L281]
> I think the problem is that the RelBuilder needs a Context or a Factory 
> passed in.  It doesn't seem meant to handle a RelBuilder itself?  
> Not sure how to fix this, but I hit this problem when I needed to use my own 
> RelBuilder containing a value within my RelBuilder.Config, and it was not 
> using my RelBuilder in this rule.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to