You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "CurtHagenlocher (via GitHub)" <gi...@apache.org> on 2023/10/19 14:44:58 UTC

Re: [PR] GH-37359: [C#] Add ToList() to Decimal128Array and Decimal256Array [arrow]

CurtHagenlocher commented on PR #37383:
URL: https://github.com/apache/arrow/pull/37383#issuecomment-1771140768

   @DanTm99 I'm curious whether you find useful to option to specify includeNulls=false. I recognize that it already exists for PrimitiveArray and think that API consistency is a valuable property, but at the same time I wonder about the kind of scenarios where this would be used.
   
   If Decimal128Array and Decimal256Array were to implement IReadOnlyList<decimal?> and ICollection<decimal?> -- allowing efficient use of LINQ ToList() -- would that meet your use case?


-- 
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