emilk opened a new issue, #8351: URL: https://github.com/apache/arrow-rs/issues/8351
The `Display` implementation for `DataType` is pretty bad for some things, including: * FixedSizeBinary * Timestamp * Struct (non-reversible / lossy) * Union * Dictionary * Decimal* * Map * RunEndEncoded We need to have a good overall design for these, and then implement it. ## Design considerations ### Readability The output should be short and readable for common cases, e.g. `List<nullable u8>` ### Reversibility We should be able to `parse` back the original `DataType`. Open question: Should that also include meta-data on any embedded `Field`:s? ### Safe Strings like field names need to be escaped to avoid string injection bugs (e.g. strings containing commas, quotes, newlines, …) ### Familiar We currently use parentheses for most datatypes, e.g. `List(u8)`. This is _fine_, but we should _consider_ using either `<>` like in C/Rust/C#/Java or even `[ ]` like in Python. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
