kgyrtkirk commented on issue #17455: URL: https://github.com/apache/druid/issues/17455#issuecomment-2467529380
is this really a regression? did it worked in earlier versions - if yes in which version? * A way to fix this in the logical optimizer way could be to recognizing the fact that `CAST` can be pulled out from `CASE` and then the same for `SUM` ...but I feel like the above could only be a rare hit. * on the other hand it seems like `AggregateCaseToFilterRule` only consideres [integer types](https://github.com/apache/calcite/blob/f347fba2d25d62c98fe3bb5f28bc9b508f7ca017/core/src/main/java/org/apache/calcite/rel/rules/AggregateCaseToFilterRule.java#L235C16-L235C28) ; right now I don't see any reason why that can't apply to `double` typed expressions I believe in 31 `case_long` will also retain the `case_searched` due to a logic issue #17378 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
