hcrosse commented on PR #1515: URL: https://github.com/apache/arrow-go/pull/1515#issuecomment-6780406164
> Why do we want to successfully parse and format Decimal128 and Decimal256 values beyond the maximum supported scale? @zeroshade honestly good question! I found this while testing my other related changes and got a little lost in the sauce so to speak. We already accept Decimals beyond the maximum scale via `NewDecimalType`/IPC/C data and currently just panic on them. I think rejecting them is the better solution though. Do you want me to change this PR to give better errors in `Decimal` or should I move the rejections earlier to when the type is created or imported? -- 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]
