nevi-me opened a new pull request, #1961:
URL: https://github.com/apache/arrow-rs/pull/1961
# Which issue does this PR close?
Closes #1882.
# Rationale for this change
See #1882
# What changes are included in this PR?
Removes the `indexmap` dependency to evaluate the impact on tests
# Are there any user-facing changes?
Yes, `arrow::json::reader::ReaderBuilder::with_format_strings` used to
incorrectly take an `IndexMap` aliased as `HashMap`, it and other related
decoder options now take a `HashMap`
_____
Note that there are other crates that depend on `indexmap`.
These are:
* clap - `integration-testing`, in `parquet` as an optional feature for
binaries
* h2 - `arrow-flight` (dependency of `tonic`)
* petgraph - `arrow-flight` `arrow-flight` (dependency of `tonic-build`)
* tower - `arrow-flight` (dependency of `tonic`)
So if we can find a suitable resolution to the JSON writer, we could remove
`indexmap` from the `arrow` crate.
I've also noticed that our write behaviour changes if we don't
`preserve_order` on serde-json, which could be a blocker.
--
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...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
