Michael Smith has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22092 )

Change subject: IMPALA-13531: (Addendum) CALCITE-5207 workaround
......................................................................


Patch Set 31:

> Patch Set 31:
>
> Been over 7 months since I looked at this.
>
> I see I had the comment that the following query failed for me:
>
> EXPLAIN with d1 as (
>   select id_col from functional.date_tbl where
>     cast('1998-08-04' as date) between date_col and (date_col + INTERVAL 14 
> days)),
> d2 as (
>   select id_col from functional.date_tbl where
>     cast('1998-08-04' as date) between date_col and (date_col + INTERVAL 14 
> days))
> select d1.id_col, d2.id_col from d1, d2;
>
> This query succeeds in the latest build, so I'm wondering again if this patch 
> is still needed.

I still run into this in the CTE test:

    E   RuntimeException: java.lang.RuntimeException: Error while applying rule 
MaterializedViewJoinRule(Filter), args 
[rel#1520:LogicalFilter.NONE(input=RelSubset#1519,condition=AND(<=($1, 
1998-08-04), >=(+($1, *(14:TINYINT, 86400000:INTERVAL DAY)), 1998-08-04)))]
    E   CAUSED BY: RuntimeException: Error while applying rule 
MaterializedViewJoinRule(Filter), args 
[rel#1520:LogicalFilter.NONE(input=RelSubset#1519,condition=AND(<=($1, 
1998-08-04), >=(+($1, *(14:TINYINT, 86400000:INTERVAL DAY)), 1998-08-04)))]
    E   CAUSED BY: ClassCastException: class 
org.apache.calcite.sql.type.BasicSqlType cannot be cast to class 
org.apache.calcite.sql.type.IntervalSqlType 
(org.apache.calcite.sql.type.BasicSqlType and 
org.apache.calcite.sql.type.IntervalSqlType are in unnamed module of loader 
'app')


--
To view, visit http://gerrit.cloudera.org:8080/22092
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9d19e83a3e2188a33d2d4a68b5afd3d457f4077c
Gerrit-Change-Number: 22092
Gerrit-PatchSet: 31
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Steve Carlin <[email protected]>
Gerrit-Comment-Date: Wed, 08 Jul 2026 18:47:30 +0000
Gerrit-HasComments: No

Reply via email to