Implement CASE flattening optimizations#34175
Merged
Merged
Conversation
Contributor
Author
|
I'll mark it as ready after adding some tests for the new transformations (they already affect some existing tests, but having some dedicated ones is probably better). |
ranma42
marked this pull request as ready for review
July 6, 2024 16:36
ranma42
marked this pull request as draft
July 6, 2024 17:13
ranma42
marked this pull request as ready for review
July 6, 2024 19:10
ranma42
force-pushed
the
factory-case
branch
2 times, most recently
from
July 7, 2024 11:32
62b6472 to
d838062
Compare
It filters out all elements after a certain predicate it satisfied.
This makes the expression a little simpler and (especially) normalizes the ELSE sub-expression.
maumar
reviewed
Jul 12, 2024
roji
reviewed
Jul 12, 2024
| { | ||
| var test = caseWhenClause.Test; | ||
|
|
||
| if (operand == null && test is CaseExpression { Operand: null, WhenClauses: [var clause] } testExpr) |
Member
There was a problem hiding this comment.
Suggested change
| if (operand == null && test is CaseExpression { Operand: null, WhenClauses: [var clause] } testExpr) | |
| if (operand == null && test is CaseExpression { Operand: null, WhenClauses: [var nestedSingleClause] } testExpr) |
| => expr is SqlConstantExpression { Value: false or null }; | ||
|
|
||
| bool IsTrue(SqlExpression expr) | ||
| => expr is SqlConstantExpression { Value: true }; |
Member
There was a problem hiding this comment.
Honestly, with pattern matching, the syntax is simple enough that maybe we should just inline, rather than have a helper method like this…
maumar
approved these changes
Jul 12, 2024
Contributor
|
nice improvement, thanks! |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
It is possible to perform most of the
CASEoptimizations at construction time.This improves the SQL emitted by SqlServer and makes it easy to implement the optimizations proposed in #18774.
Fixes #18774.
Fixes #18935.