thisisnic commented on a change in pull request #11758:
URL: https://github.com/apache/arrow/pull/11758#discussion_r754531698
##########
File path: r/R/type.R
##########
@@ -187,8 +187,11 @@ NestedType <- R6Class("NestedType", inherit = DataType)
#' @param timezone For `timestamp()`, an optional time zone string.
#' @param byte_width byte width for `FixedSizeBinary` type.
#' @param list_size list size for `FixedSizeList` type.
-#' @param precision For `decimal()`, precision
-#' @param scale For `decimal()`, scale
+#' @param precision For `decimal()`, precision. The number of significant
digits
+#' the arrow `decimal` type can represent. Currently `decimal()` is mapped
+#' to `DecimalType128`, having a maximum precision of 38 significant digits.
+#' @param scale For `decimal()`, scale. The number of digits after the decimal
Review comment:
```suggestion
#' @param scale For `decimal()`, the number of digits after the decimal
```
--
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]
