[GitHub] [arrow] dragosmg commented on a change in pull request #11651: ARROW-14558 [R] clarify OOP system wording in the Arrow vignette

[GitBox <gi...@apache.org>]

dragosmg commented on a change in pull request #11651:
URL: https://github.com/apache/arrow/pull/11651#discussion_r745662383



##########
File path: r/vignettes/arrow.Rmd
##########
@@ -216,7 +216,7 @@ Note that the `attributes()` stored in `$metadata$r` are only understood by R. I
 
 ## Class structure and package conventions
 
-C++ is an object-oriented language, so the core logic of the Arrow library is encapsulated in classes and methods. In the R package, these classes are implemented as `R6` reference classes, most of which are exported from the namespace.
+C++ is an object-oriented language, so the core logic of the Arrow library is encapsulated in classes and methods. In the R package, these classes are implemented using the `R6` object-oriented programming system, and most of them are exported from the namespace.

Review comment:
       Should we also link to Hadley's R6 (sub)chapter in _Advanced R_? Or would anyone interested find that as it's not really a hidden resource?
    




-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org