sunchao commented on pull request #7751:
URL: https://github.com/apache/arrow/pull/7751#issuecomment-658968604


   > Date64 is an arrow specific format, and thus to test it we need to have a 
file in parquet with that format, since we do not have a writer yet to play 
around.
   
   I don't fully understand this. You mean we need Parquet file with the Date64 
type? 
   
   To my understanding, there is no conversion from Parquet type to Arrow's 
Date64 type, therefore the conversion clause may never happen. See 
[here](https://github.com/apache/arrow/blob/master/rust/parquet/src/arrow/schema.rs#L409)
 for Rust and 
[here](https://github.com/apache/arrow/blob/master/cpp/src/parquet/arrow/schema_internal.cc#L161)
 for C++


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to