nuno-faria commented on PR #17232: URL: https://github.com/apache/datafusion/pull/17232#issuecomment-3217094892
Thanks for the changes @BlakeOrth. I also believe that automatically resolving the partitioning is the best approach. Like @alamb said, it would be nice to document and add an option to toggle this off if needed. In addition to adding documentation at https://datafusion.apache.org/user-guide/sql/ddl.html#create-external-table, I think we should also have additional documentation here: https://datafusion.apache.org/user-guide/cli/datasources.html#create-external-table. One thing to note is that I believe that `CREATE EXTERNAL TABLE` now has a different behavior from `SessionContext::register_parquet`, which still requires partitions to be manually specified. I also feel `register_parquet` could use a form of automatic partition detection, maybe in a flag passed to `ParquetReadOptions`. -- 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
