[GitHub] [arrow] jonkeane commented on a change in pull request #12062: ARROW-15173: [R] Provide backward compatibility for bridge to older versions of pyarrow

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

jonkeane commented on a change in pull request #12062:
URL: https://github.com/apache/arrow/pull/12062#discussion_r781524311



##########
File path: r/R/python.R
##########
@@ -224,3 +306,14 @@ install_pyarrow <- function(envname = NULL, nightly = FALSE, ...) {
     reticulate::py_install("pyarrow", envname = envname, ...)
   }
 }
+
+pyarrow_version <- function() {
+  pa <- reticulate::import("pyarrow")
+  version_string <- pa$`__version__`
+  # remove trailing .devXXX because it won't work with package_version()
+  package_version(gsub("\\.dev.*?$", "", version_string))
+}
+
+pyarrow_version_pointers_changed <- function() {
+  "7.0.0"
+}

Review comment:
       Is there any reason you have this as a function returning a version instead of simply `pyarrow_version_pointers_changed <- "7.0.0"`?




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