szehon-ho opened a new pull request, #50851:
URL: https://github.com/apache/spark/pull/50851

   
   
   ### What changes were proposed in this pull request?
   Limit the tests added in https://github.com/apache/spark/pull/50701 and 
https://github.com/apache/spark/pull/50593   only for ANSI_ENABLED mode.
   
   ### Why are the changes needed?
   These tests fail in non-ANSI mode.  The reason is that 
https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/V2ExpressionUtils.scala
 only converts majority of catalyst => V2 expressions in ANSI mode.  So , we do 
not get any V2Expression in non-ANSI case.
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   Existing unit test run with SPARK_ANSI_SQL_MODE=false
   
   
   ### Was this patch authored or co-authored using generative AI tooling?
   No
   


-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to