surendralilhore opened a new pull request, #12035: URL: https://github.com/apache/gluten/pull/12035
Starting from Spark 4.1, [SQLConf.get.getConf(SQLConf.MAP_KEY_DEDUP_POLICY)](https://github.com/apache/spark/blob/3c1a8423d8d330facbfd25b55c80fc01c32f357c/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala#L5277) returns an enum value instead of a String. The existing comparison against the `.toString` of the `EXCEPTION` variant would always evaluate to not-equal (comparing enum != String), incorrectly rejecting `str_to_map` offloading even when the policy is set to `EXCEPTION`. Add `.toString` on the left side so both operands are compared as Strings, restoring correct behavior across all Spark versions. ## What changes are proposed in this pull request? Fix the code but to make code compatible with Spark 4.1 ## How was this patch tested? Added new UT test to verify the fix. ## Was this patch authored or co-authored using generative AI tooling? GitHub Copilot CLI (powered by Claude Opus 4.6) -- 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]
