crepererum commented on code in PR #16783: URL: https://github.com/apache/datafusion/pull/16783#discussion_r2207466414
########## datafusion/sqllogictest/test_files/parquet.slt: ########## @@ -130,8 +130,7 @@ STORED AS PARQUET; ---- 3 -# Check output plan again, expect no "output_ordering" clause in the physical_plan -> ParquetExec, -# due to there being more files than partitions: +# Check output plan again Review Comment: I think the actual reason why the `output_ordering` was missing here wasn't the number of files, but because DF had trouble with a column that wasn't marked as `NOT NULL` (i.e. is nullable). -- 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
