You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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 : #1971 , Issue : #532
Expected behavior
ANSI mode should be supported and the override function can be removed since it is redundant