CurtHagenlocher commented on code in PR #38508: URL: https://github.com/apache/arrow/pull/38508#discussion_r1376249714
########## csharp/test/Apache.Arrow.Tests/DecimalUtilityTests.cs: ########## Review Comment: Yes, I didn't think it made sense either to change the behavior of the existing functions or to make "policy" kinds of decisions best left to the application. For instance, I can imagine an application saying for a decimal128 that "if the declared precision and scale always fit into a System.Decimal then return System.Decimal otherwise return SqlDecimal. -- 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]
