Skip to content

Commit

Permalink
Remove unneeded rc feature of serde (#990)
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)
  • Loading branch information
carols10cents authored and alamb committed Dec 9, 2021
1 parent a92672e commit 81951af
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion arrow/Cargo.toml
Original file line number Diff line number Diff line change
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 81951af

Please sign in to comment.