cloud-fan commented on PR #46013: URL: https://github.com/apache/spark/pull/46013#issuecomment-2050899887
Yea there are still some gaps between Spark ANSI mode and the SQL standard. But mostly ANSI is still a better default than no ANSI at all. [SPARK-46374](https://issues.apache.org/jira/browse/SPARK-46374) is a tricky one. We do realize the index should be 1-based and it's the behavior for new functions like `ElementAt`. The `arr[idx]` syntax has been there for so long and it's really scaring if we change the result silently by making it 1-based. -- 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
