alamb commented on PR #11524: URL: https://github.com/apache/datafusion/pull/11524#issuecomment-2234289349
> This PR is defining the current behavior only. > Not defining what it should be (altho yes the test cases use "should" lingo 😆 ). I changed the title to reflect what this PR does 🙏 (as that shows up as the final commit message in the logs) In general it is my opinion that the DataFusion defaults should mirror the arrow defaults -- I believe that was the intention when @devinjdangelo made the first version of this code. Over time the defaults in arrow have evolved and DataFusion hasn't kept up. -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
