DanTm99 commented on PR #37383: URL: https://github.com/apache/arrow/pull/37383#issuecomment-1771267096
@CurtHagenlocher yes, the use case I had for this would even be met by `Decimal128Array` and `Decimal256Array` just implementing `IEnumerable<decimal?>`, as all I need to do is use them with LINQ. The reason I just added these `ToList()` methods was because it would meet my use case while also making the classes more consistent with `PrimitiveArray<T>`. However, implementing `IReadOnlyList<decimal?>` and ICollection<decimal?>` should be done as part of a separate change, as this change was moreso to bring more consistency to this API. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
