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

Thomas Rebele commented on CALCITE-5686:
----------------------------------------

Sorry, I've used a workaround, which was better suited to the problem at the 
end. I hope I'll find some time soon to work on the remaining Calcite tickets 
that are assigned to me.

> Generalize return types of the linq4j expression shuttle
> --------------------------------------------------------
>
>                 Key: CALCITE-5686
>                 URL: https://issues.apache.org/jira/browse/CALCITE-5686
>             Project: Calcite
>          Issue Type: Improvement
>            Reporter: Thomas Rebele
>            Assignee: Thomas Rebele
>            Priority: Minor
>
> The following block in org.apache.calcite.linq4j.tree.Shuttle prevents 
> replacing constants with other expressions:
> {code:java}
>   public ConstantExpression visit(ConstantExpression constantExpression) {
>     return constantExpression;
>   } {code}
> The return type of the corresponding accept function is more general:
> {code:java}
> @Override public Expression accept(Shuttle shuttle) {
>     return shuttle.visit(this);
>   } {code}



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

Reply via email to