paleolimbot commented on code in PR #9077:
URL: https://github.com/apache/arrow-rs/pull/9077#discussion_r2658683520
##########
parquet/src/arrow/arrow_writer/mod.rs:
##########
@@ -4163,68 +4167,6 @@ mod tests {
}
}
- #[test]
- fn test_arrow_writer_explicit_schema() {
Review Comment:
I think it should be fine as well given that the `geospatial.rs` test still
passes. The intent was to test that the code path invoked by
`with_parquet_schema()` actually resulted in the Parquet schema being used by
the writer. If that code path now results in something else happening than what
I wrote here, it's probably best to keep the test but adjust the expected
result (i.e., this test is just checking for plugged in wires).
--
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]
