zhuqi-lucas commented on PR #16599: URL: https://github.com/apache/datafusion/pull/16599#issuecomment-3015303000
> > Now it default to false, but i am not sure if it will make other things broken. > > Yeah I think it will break other things -- it isn't correct in general to treat binary columns as strings -- the reason it is needed for the clickbench_partitioned dataset is that the parquet files in that dataset were written with some ancient version of parquet-cpp and didn't set the type annotations correctly. Interestingly hits.parquet (the single file) doesn't have the same problem Good explain, thank you @alamb , i also found that hits.parquet doesn't have the same problem, that's very interesting. -- 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
