paleolimbot commented on a change in pull request #11521:
URL: https://github.com/apache/arrow/pull/11521#discussion_r753336995



##########
File path: r/vignettes/install.Rmd
##########
@@ -85,33 +91,52 @@ Alternatively, you can set
 export LIBARROW_MINIMAL=false
 ```
 
-to build the Arrow libraries from source with optional features such as 
compression libraries
-enabled. This will increase the build time but provides many useful features.
+to build the Arrow libraries from source with optional features such as 
compression libraries enabled. This will increase the build time but provides 
many useful features.

Review comment:
       Again, maybe the latest PR enabling Snappy and lz4 by default makes this 
not necessary?

##########
File path: r/vignettes/install.Rmd
##########
@@ -41,13 +41,17 @@ release of the R package along with any C++ dependencies.
 
 # Installation basics
 
+## Install the latest release
+

Review comment:
       Is it worth explaining why `NOT CRAN = TRUE` here? (Or maybe the latest 
PR enabling Snappy and lz4 makes this env var not necessary?)




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