xiedeyantu commented on PR #4471: URL: https://github.com/apache/calcite/pull/4471#issuecomment-3101692314
> I add a Quidem test. But the case-style Aggregate is not considered the best plan (its cost is the same as the true filtered Aggregate, so the best node of RelSubSet is not updated). I think I can only use sql to test the results of case-style and true filtered-style separately. @xiedeyantu I think you can verify this rule in `RelToSqlConverterTest.java`, which can convert the `agg filter` into SQL in the form of `case when`, and then verify the SQL before and after the conversion in the `.iq` test. -- 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]
