Revise FromIterator for Decimal128Array to use Into instead of Borrow #2441
Labels
enhancement
Any new improvement worthy of a entry in the changelog
parquet
Changes to the parquet crate
Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Currently the element type of
FromIterator
forDecimal128Array
isBorrow<Option<i128>
. ButBorrow<Option<i128>
is hard to extend to other types e.g.Decimal128
. Because the rust compiler doesn't allow us to implementBorrow<Option<i128>
forOption<Decimal128>
asOption
is not defined in this crate.Using
Into
trait seems more suitable and extensible.Describe the solution you'd like
Describe alternatives you've considered
Additional context
The text was updated successfully, but these errors were encountered: