coderfender opened a new issue, #2555: URL: https://github.com/apache/datafusion-comet/issues/2555
### Describe the bug I was browsing through ANSI epic and saw that ANSI support for Remainder function is disabled despite having support (and corresponding tests) for it on the scala and native side. My hunch is that it might have been missed while refactoring `QueryPlanSerde` class ### Steps to reproduce `CometRemainder` class has `getSupportLevel` overridden to not support ANSI mode while there is indeed support on native + scala side (including tests) PR : https://github.com/apache/datafusion-comet/pull/1971 , Issue : https://github.com/apache/datafusion-comet/issues/532 ### Expected behavior ANSI mode should be supported and the override function can be removed since it is redundant ### Additional context _No response_ -- 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]
