ByteBaker commented on code in PR #10304:
URL: https://github.com/apache/arrow-rs/pull/10304#discussion_r3564988248
##########
arrow-schema/src/datatype.rs:
##########
@@ -361,12 +361,12 @@ pub enum DataType {
///
/// For example the number 123.45 has precision 5 and scale 2.
///
- /// In certain situations, scale could be negative number. For
- /// negative scale, it is the number of padding 0 to the right
- /// of the digits.
+ /// In certain situations, scale can be negative. Negative scale
+ /// rounds values to the left of the decimal point, and precision is
+ /// the maximum number of non-rounded digits.
///
- /// For example the number 12300 could be treated as a decimal
- /// has precision 3 and scale -2.
+ /// For example, with precision 2 and scale -3, the value 12345 is
+ /// rounded to 12000 and represented as the stored integer 12.
Review Comment:
I updated it. TBH I find explaining this somewhat confusing by nature. 😬
But please see if the newer one is more clearer.
--
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]
