Skip to content

Propagate allowOptimizedExpansion to CASE results#34304

Merged
maumar merged 2 commits into
dotnet:mainfrom
ranma42:optimize-case-then
Jul 29, 2024
Merged

Propagate allowOptimizedExpansion to CASE results#34304
maumar merged 2 commits into
dotnet:mainfrom
ranma42:optimize-case-then

Conversation

@ranma42

@ranma42 ranma42 commented Jul 28, 2024

Copy link
Copy Markdown
Contributor

If the CASE is used in a context that allows optimizations (predicates, where
NULL and FALSE are equivalent), the same holds for each of its results.

Contributes to #16050

ranma42 added 2 commits July 28, 2024 11:17
If the `CASE` is used in a context that allows optimizations (predicates, where
NULL and FALSE are equivalent), the same holds for each of its results.
@ranma42

ranma42 commented Jul 28, 2024

Copy link
Copy Markdown
Contributor Author

Found while investigating the test changes in #34172 😇

@maumar maumar merged commit ac1b366 into dotnet:main Jul 29, 2024
@maumar

maumar commented Jul 29, 2024

Copy link
Copy Markdown
Contributor

very nice improvement, thanks again @ranma42 !

@ranma42 ranma42 deleted the optimize-case-then branch July 29, 2024 19:08
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.

2 participants