apache / apache/arrow-rs

Unify `Field`'s `Display`

Open
#8,775 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Rust
Stars
3.6k
Forks
1.3k
Avg merge
2d 18h
Merged PRs (30d)
169

Description

**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**

There are two different representations for `Field`:

1. In `Struct`, `Map`, `Union`, `RunEndEncoded`, for example: `"value": nullable Int32`
2. In `List`-like, for example: `nullable Int32, field: 'value'`

**Describe the solution you'd like**

We should use one `Display` for `Field`.

**Describe alternatives you've considered**

None.

**Additional context**

We already had `fn format_field` (this is used in `Struct`, etc.) :
https://github.com/apache/arrow-rs/blob/b8a192696b8216878f247b6a2d8dd63a09558063/arrow-schema/src/datatype_display.rs#L33

We can replace `List`-like display using the above `format_field`:
https://github.com/apache/arrow-rs/blob/b8a192696b8216878f247b6a2d8dd63a09558063/arrow-schema/src/datatype_display.rs#L119-L122

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.