[GitHub] [arrow-rs] HaoYang670 opened a new issue, #1985: Move the `DecimalArray` to a new file.

[GitBox <gi...@apache.org>]

HaoYang670 opened a new issue, #1985:
URL: https://github.com/apache/arrow-rs/issues/1985

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   The `DecimalArray` is currently defined in `array_binary.rs`, which doesn't make sense to me.
   Based on the definition : https://github.com/apache/arrow/blob/master/format/Schema.fbs#L181-L195
   Decimal Array is a logical type whose physical layout is more like a primitive array.
   
   This is the reason of moving Decimal Array out `array_binary.rs`.
   
   **Describe the solution you'd like**
   Create a new file `array_decimal.rs` and move the decimal array here.
   
   **Describe alternatives you've considered**
   We could not do this.
   
   **Additional context**
   Add any other context or screenshots about the feature request here.
   


-- 
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-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-rs] tustvold closed issue #1985: Move the `DecimalArray` to a new file.

[GitBox <gi...@apache.org>]

tustvold closed issue #1985: Move the `DecimalArray` to a new file.
URL: https://github.com/apache/arrow-rs/issues/1985


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org