Skip to content

Commit

Permalink
Remove unneeded rc feature of serde (#990) (#1016)
Browse files Browse the repository at this point in the history
Fixes #989.

This feature opts into impls for `Rc` and `Arc`, but none of the data
structures that use Serialize/Deserialize actually contain `Rc` or
`Arc`s.

See:

- [Serde docs](https://serde.rs/feature-flags.html#-features-rc)
- [PR adding this](apache/arrow#3016)

Co-authored-by: Carol (Nichols || Goulding) <193874+carols10cents@users.noreply.github.com>
  • Loading branch information
alamb and carols10cents committed Dec 9, 2021
1 parent b28385e commit 557fc11
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion arrow/Cargo.toml
Expand Up @@ -36,7 +36,7 @@ name = "arrow"
path = "src/lib.rs"

[dependencies]
serde = { version = "1.0", features = ["rc"] }
serde = { version = "1.0" }
serde_derive = "1.0"
serde_json = { version = "1.0", features = ["preserve_order"] }
indexmap = "1.6"
Expand Down

0 comments on commit 557fc11

Please sign in to comment.