pitrou commented on PR #43957:
URL: https://github.com/apache/arrow/pull/43957#issuecomment-2331116181

   > Following the same pattern, this change means that using 
`decimal(precision, scale)` instead of the specific 
`decimal32`/`decimal64`/`decimal128`/`decimal256` functions results in the 
following functionality
   
   I'm afraid this may massively break user code. I would suggest another 
approach:
   * deprecate the `decimal()` factory while keeping its current behavior of 
always returning at least `decimal128`
   * introduce a new `smallest_decimal()` factory that is documented to return 
the smallest possible type, and explicitly makes no guarantees about the 
stability of the return type
   


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

Reply via email to