thisisnic commented on code in PR #46720:
URL: https://github.com/apache/arrow/pull/46720#discussion_r2145340640


##########
r/vignettes/data_types.Rmd:
##########
@@ -96,26 +122,38 @@ Note that the Arrow specification also permits 
half-precision (16 bit) floating
 
 ## Fixed point decimal types
 
-Arrow also contains `decimal()` data types, in which numeric values are 
specified in decimal format rather than binary. Decimals in Arrow come in two 
varieties, a 128 bit version and a 256 bit version, but in most cases users 
should be able to use the more general `decimal()` data type function rather 
than the specific `decimal128()` and `decimal256()` functions. 
+Arrow also contains `decimal()` data types, in which numeric values are 
specified in decimal format rather than binary.
+Decimals in Arrow come in two varieties, a 128 bit version and a 256 bit 
version, but in most cases users should be able 
+to use the more general `decimal()` data type function rather than the 
specific `decimal32()`, `decimal64()`, `decimal128()`,

Review Comment:
   I split the text up in this vignette to make it easier to edit and track 
edits in future, but the only actual changes I made were to this line, where I 
added the new decimal types in.



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