huaxingao commented on code in PR #11257: URL: https://github.com/apache/iceberg/pull/11257#discussion_r1794322082
########## spark/v4.0/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewriteDataFilesAction.java: ########## @@ -138,6 +138,7 @@ public class TestRewriteDataFilesAction extends TestBase { public static void setupSpark() { // disable AQE as tests assume that writes generate a particular number of files spark.conf().set(SQLConf.ADAPTIVE_EXECUTION_ENABLED().key(), "false"); + spark.conf().set(SQLConf.ANSI_ENABLED().key(), "false"); Review Comment: I actually meant to change only the ANSI setting for `TestRewriteDataFilesAction#testZOrderAllTypesSort`, because CAST(longCol AS BINARY) is not allowed with ANSI, which is the default now for Spark 4.0. I fixed the test to use ANSI false for testZOrderAllTypesSort. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org