apache / apache/datafusion

Change `serde` dependency to `serde_core` where applicable

Open
#17,786 3 comments 1 reaction 1 assignee Claimed by @AdamGS View on GitHub
enhancement
Dominant language
Rust
Stars
9.3k
Forks
2.4k
Avg merge
3d 7h
Merged PRs (30d)
344

Description

### Is your feature request related to a problem or challenge?

Currently some datafusion crates depend on `serde`, which may take a long time to compile if -- due to feature unification -- datafusion users opted into the `derive` features. See https://github.com/serde-rs/serde/pull/2608 . However we often don't need `derive` and hence we could depend on the actual core part of serde called `serde_derive`.

### Describe the solution you'd like

Check our dependencies and where we don't use `derive`, change the dependency from `serde` to `serde_derive`.

### Describe alternatives you've considered

\-

### Additional context

- https://github.com/serde-rs/serde/pull/2608
- https://github.com/apache/arrow-rs/issues/8451

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.