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

Jiajun Xie commented on CALCITE-6088:
-------------------------------------

Fixed in 
[94dc167.|https://github.com/apache/calcite/commit/94dc1673da824174f9271677ead73cfae1aeb29b]

Thanks [~MasseGuillaume]  and [~mbudiu].

> SqlItemOperator fails in RelToSqlConverter
> ------------------------------------------
>
>                 Key: CALCITE-6088
>                 URL: https://issues.apache.org/jira/browse/CALCITE-6088
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.35.0
>            Reporter: Guillaume Massé
>            Assignee: Jiajun Xie
>            Priority: Major
>              Labels: pull-request-available
>
> We have an expression that looks like:
> {code:java}
> foo[1].bar {code}
> When we run RelToSqlConverter we get the following strack trace: (Calcite 
> 1.35.0)
>  
> {code:java}
> [info]   java.lang.ClassCastException: class 
> org.apache.calcite.sql.SqlBasicCall cannot be cast to class 
> org.apache.calcite.sql.SqlIdentifier (org.apache.calcite.sql.SqlBasicCall and 
> org.apache.calcite.sql.SqlIdentifier are in unnamed module of loader 
> sbt.internal.FlatLoader @3de98cb6)
> [info]   at 
> org.apache.calcite.rel.rel2sql.SqlImplementor$Context.toSql(SqlImplementor.java:676)
> [info]   at 
> org.apache.calcite.rel.rel2sql.RelToSqlConverter.visit(RelToSqlConverter.java:458)
> [info]   at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> [info]   at 
> java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
> [info]   at 
> java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
> [info]   at java.base/java.lang.reflect.Method.invoke(Method.java:568)
> [info]   at org.apache.calcite.util.ReflectUtil$2.invoke(ReflectUtil.java:532)
> [info]   at 
> org.apache.calcite.rel.rel2sql.RelToSqlConverter.dispatch(RelToSqlConverter.java:143)
> [info]   at 
> org.apache.calcite.rel.rel2sql.RelToSqlConverter.visitInput(RelToSqlConverter.java:151)
> [info]   at 
> org.apache.calcite.rel.rel2sql.SqlImplementor.visitInput(SqlImplementor.java:220)
>  {code}
>  
> This is because there is a missing case for the function
> {code:java}
> public SqlNode toSql(@Nullable RexProgram program, RexNode rex) { 
> // ...
>         switch (rex.getKind()){
>         // ...
>         case ITEM: // << missing
>         // missing impl         
>         default:
>         sqlIdentifier = (SqlIdentifier) toSql(program, referencedExpr);
> }{code}
>  
> ```
> The rex kind is ITEM
>  



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

Reply via email to