alamb commented on issue #15780: URL: https://github.com/apache/datafusion/issues/15780#issuecomment-2976060769
> variability comes from reading the data, parsing parquet, etc. and once it's in memory as arrow converting from Int32 to Int64 is tri This makes a lot of sense to me > It only deals with the View types, which I think are a special case of the general idea. I agree -- this is a nice observation > What I am thinking is: > For any cases where we can read from parquet directly into the table type do that (e.g. StringView and Utf8 are stored the sa me in Parquet, both UInt8 and UInt16 are stored as INT32 in Parquet). This is basically what is happening with the view types already. > If a cast is required prefer to cast the filter / scalar values or the smaller column (we can use similar logic as what we use to reorder filters). This also makes sense to me I still think the major / critical usecase is going to be "extract a field from this struct / variant" but anything else we do will be bonus -- 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