forked from apache/calcite
-
Notifications
You must be signed in to change notification settings - Fork 53
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
KE-11534 Upgrade Calcite to 1.30 and adapt it for Kylin #320
Merged
gleonSun
merged 42 commits into
Kyligence:kycalcite-1.30.0.x
from
gleonSun:calcite-guava
Mar 11, 2024
Merged
KE-11534 Upgrade Calcite to 1.30 and adapt it for Kylin #320
gleonSun
merged 42 commits into
Kyligence:kycalcite-1.30.0.x
from
gleonSun:calcite-guava
Mar 11, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
For SDK dialect conversion Calcite 1.30 changed Rexcall operator from SqlCaseOperator to SqlPostfixOperator in RelOptUtil#isDistinctFromInternal Calcite 1.30 remove EquiJoinInfo and NonEquiJoinInfo, but we need rexbuilder Calcite 1.30 implement CHAR function Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions Revert "Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions" This reverts commit 27f1a28. [Follow up] Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions Refer to KE-36291 for an adapted method to simplify the filter condition Refer to AL-5295 fix the ut about CharNColumnTest#testCharNColumn Calcite 1.30 fix about SumCaseWhenFunctionRule and CountDistinctCaseWhenFunctionRule [Follow up] Fix ProjectRel replacement and old agg replacement caused by RelBuilder#aggregate method Roll back Calcite's computeDigest and simplifyCase logic from previous versions to ensure correct matching of Kylin models [Follow up] Fix the way digest is calculated, delete the previous logic Fix exception in Calcite 1.30 where data type conversion affected logical plan changes Fix with Sort optimize and Trim function Revert "Calcite 1.30 implement CHAR function" This reverts commit 0e1f6b0. [CALCITE-5241] Implement CHAR function for MySQL and Spark, also JDBC '{fn CHAR(n)}' Close apache#2878 Debug for Calcite deploy Calcite 1.30 don't keep the precision of BigDecimal, add scale to fix exception Fix SqlBasicCall's Deep Copy Logic Raises Rule Optimization Exception and TDVT test Change kap-external-guava20 to kylin-external-guava30 and package name Remove the default Unicode operation on the quoteStringLiteral function in Calcite 1.30
…Calcite’s JoinInfo
…ts Deprecated flag and remove the RexBuilder#kylinMakeZeroLiteral method
… RelBuilder#groupKey
…sistency with the latest operandList references
…Identifier#getComponentParserPosition
…ng scale in a less invasive way
…bstractParserImpl. Instead, adjust Kylin to fit
…types in Calcite StandardConvertletTable#convertOperands while adapting in Kylin
…lder in Calcite’s JoinInfo" This reverts commit 50d862e.
gleonSun
force-pushed
the
calcite-guava
branch
from
August 25, 2023 06:54
50be7a4
to
02cb750
Compare
…ion [CALCITE-5145] CASE statement within GROUPING SETS throws type mis-match exception Co-authored-by: yingyuwang <[email protected]>
…eldTrimmer to align with the community
… use of dynamic parameter exception
…avoid incorrect null vs true/false comparisons
…l for replacing cc
pfzhan
force-pushed
the
calcite-guava
branch
2 times, most recently
from
December 12, 2023 09:30
93199da
to
44cbc10
Compare
support ifnull
pfzhan
force-pushed
the
calcite-guava
branch
from
December 12, 2023 12:06
44cbc10
to
ac5f758
Compare
support ifnull
…is non-numeric Close apache#3038 Co-authored-by: Bruce Irschick <[email protected]>
gleonSun
force-pushed
the
calcite-guava
branch
2 times, most recently
from
March 11, 2024 07:50
832e969
to
6eb6b2d
Compare
eventd
approved these changes
Mar 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For SDK dialect conversion
Calcite 1.30 changed Rexcall operator from SqlCaseOperator to SqlPostfixOperator in RelOptUtil#isDistinctFromInternal
Calcite 1.30 remove EquiJoinInfo and NonEquiJoinInfo, but we need rexbuilder
Calcite 1.30 implement CHAR function
Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions
Revert "Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions"
This reverts commit 27f1a28.
[Follow up] Fallback the logic of the simplification condition to version 1.16, otherwise it may lead to a failure to prune partitions
Refer to KE-36291 for an adapted method to simplify the filter condition
Refer to AL-5295 fix the ut about CharNColumnTest#testCharNColumn
Calcite 1.30 fix about SumCaseWhenFunctionRule and CountDistinctCaseWhenFunctionRule
[Follow up] Fix ProjectRel replacement and old agg replacement caused by RelBuilder#aggregate method
Roll back Calcite's computeDigest and simplifyCase logic from previous versions to ensure correct matching of Kylin models
[Follow up] Fix the way digest is calculated, delete the previous logic
Fix exception in Calcite 1.30 where data type conversion affected logical plan changes
Fix with Sort optimize and Trim function
Revert "Calcite 1.30 implement CHAR function"
This reverts commit 0e1f6b0.
[CALCITE-5241] Implement CHAR function for MySQL and Spark, also JDBC '{fn CHAR(n)}'
Close apache#2878
Debug for Calcite deploy
Calcite 1.30 don't keep the precision of BigDecimal, add scale to fix exception
Fix SqlBasicCall's Deep Copy Logic Raises Rule Optimization Exception and TDVT test
Change kap-external-guava20 to kylin-external-guava30 and package name
Remove the default Unicode operation on the quoteStringLiteral function in Calcite 1.30