nbenn opened a new pull request, #954: URL: https://github.com/apache/arrow-nanoarrow/pull/954
## Summary - Add an `as_nanoarrow_array()` method for `AsIs` that strips the class and dispatches again, mirroring the `infer_nanoarrow_schema()` method. An `I()`-wrapped base vector, such as a list column kept whole in a `data.frame()`, no longer falls through to the arrow fallback. - The method does not forward `...`: when the C conversion calls back for a data frame column, `...` carries `.from_c = TRUE`, which would still route the unwrapped value to arrow. - Values whose class has its own method, such as `I(as.Date(...))` or an `I()`-wrapped blob, convert as before, with or without arrow. An `I()`-wrapped matrix, which failed even with arrow installed, and an `I()`-wrapped `integer64` with missing values, which needed arrow, now convert too. - The test sets the `nanoarrow.without_arrow` option, so it fails on main even where arrow is installed, as it is in CI. Fixes #953 -- 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]
