singhpratech opened a new pull request, #51236:
URL: https://github.com/apache/arrow/pull/51236

   ### Rationale for this change
   
   Follows up on #39793. `Array$export_to_c()` and `Array$import_from_c()` 
work, but the functions that
   produce the pointers they take, `allocate_arrow_schema()` and friends, are 
not exported, and none of
   it has a help page, so a package that hands a column to another library over 
the C Data Interface
   has to use `arrow:::`, which `R CMD check` flags.
   
   ### What changes are included in this PR?
   
   The six allocate/delete functions from the issue are exported and documented 
on one help page
   (`?c-data-interface`) with a round-trip example; the `Array` methods list 
names `$export_to_c()`
   and `$import_from_c()`; a NEWS entry; two tests in test-bridge.R. NAMESPACE 
and the man pages were
   regenerated with roxygen2 8.0.0, the version in `Config/roxygen2/version`, 
so the diff is only the
   additions. Per @thisisnic's note on #51227, the alignment behaviour of 
`Array$create()` is a code
   comment in the new file, not user documentation.
   
   ### Are these changes tested?
   
   Yes: `test-bridge.R` gains a test that the six names are exported and a test 
that an `Array` with
   a null round-trips through the exported functions. The example on the help 
page runs under
   `R CMD check` as well.
   
   ### Are there any user-facing changes?
   
   Yes: six newly exported functions and one new help page. No existing 
behaviour changes.
   
   Thank you for looking at this. Happy to adjust the wording of the help page 
or move the
   functions' documentation wherever fits the package's conventions better.
   
   Closes #39793.
   


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