Skip to content

Error in calcite#11447

Closed
gortiz wants to merge 2 commits intoapache:masterfrom
gortiz:errorInCalcite
Closed

Error in calcite#11447
gortiz wants to merge 2 commits intoapache:masterfrom
gortiz:errorInCalcite

Conversation

@gortiz
Copy link
Contributor

@gortiz gortiz commented Aug 28, 2023

This query fails due to a strange index error in calcite.

I found it while trying to simplify the query in #11420. They may or may not be related

@codecov-commenter
Copy link

Codecov Report

Merging #11447 (22f7a7d) into master (399f033) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #11447      +/-   ##
============================================
- Coverage     62.99%   62.98%   -0.01%     
  Complexity     1094     1094              
============================================
  Files          2302     2302              
  Lines        124025   124025              
  Branches      18901    18901              
============================================
- Hits          78126    78117       -9     
- Misses        40351    40359       +8     
- Partials       5548     5549       +1     
Flag Coverage Δ
integration ?
integration1 ?
integration2 ?
java-11 67.48% <ø> (+4.52%) ⬆️
java-17 67.33% <ø> (+4.48%) ⬆️
java-20 62.85% <ø> (-0.01%) ⬇️
temurin 62.98% <ø> (-0.01%) ⬇️
unittests 62.98% <ø> (-0.01%) ⬇️
unittests1 67.52% <ø> (ø)
unittests2 14.47% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 6 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@xiangfu0
Copy link
Contributor

This is majorly the type issue related to casting issue for identifier, fix is here: #11457

@xiangfu0
Copy link
Contributor

close as fixed #11457

@xiangfu0 xiangfu0 closed this Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants