alamb commented on code in PR #8524:
URL: https://github.com/apache/arrow-rs/pull/8524#discussion_r2662369570
##########
parquet/src/arrow/arrow_writer/mod.rs:
##########
@@ -490,6 +498,18 @@ impl ArrowWriterOptions {
..self
}
}
+
+ /// Explicitly specify the Parquet schema to be used
Review Comment:
I think one rationale was to put the appropriate metadata on the field (so
the parquet writer knew what logical type to add), but I make be mistaken
I don't fully understand the concerns about type coercion, but at least part
of this API I think is designed to allow interoperability between other arrow
implementations (aka not just reading back arrays that were written in Rust,
but writing arrays that other writers will accept)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
