You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "yutannihilation (via GitHub)" <gi...@apache.org> on 2025/12/21 06:34:50 UTC
[I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
yutannihilation opened a new issue, #836:
URL: https://github.com/apache/arrow-nanoarrow/issues/836
I see this announcement on the R-devel mailing list.
> As part of clarifying and tightening the R API for packages the
non-API entry points `ATTRIB` and `SET_ATTRIB` will be removed from
installed header files and, where possible, hidden from package
access.
> Using these functions is will now result in check NOTEs. These check NOTEs will
soon be replaced by WARNINGs; this will probably happen by the end of
January.
https://stat.ethz.ch/pipermail/r-devel/2025-December/084295.html
nanoarrow (and arrow) uses `ATTRIB` only in this location to retrieve the first attribute of the ALTREP class object.
https://github.com/apache/arrow-nanoarrow/blob/e0480fed53f8db3e547db5e414528a65ebdeab4f/r/src/altrep.h#L34
I think this can be replaced with `Rf_getAttrib()` if the attribute name is known.
But..., I think these attributes are set here and this probably doesn't have the name...? (I might be wrong here)
https://github.com/r-devel/r-svn/blob/344297b51f32eba53d853cbc171d7225446626c6/src/main/altrep.c#L38-L39
If so, they say we can use `R_mapAttrib()`, which iterates over the attributes.
https://cran.r-project.org/doc/manuals/r-devel/R-exts.html#Working-with-attributes
At the moment, I don't come up with any nice fix for this.
--
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: issues-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on issue #836:
URL: https://github.com/apache/arrow-nanoarrow/issues/836#issuecomment-3701319848
Thank you for tracking that!
--
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
Re: [I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot commented on issue #836:
URL: https://github.com/apache/arrow-nanoarrow/issues/836#issuecomment-3680384110
Thank you for this!
I think I copied that bit of code from Arrow. It probably is a bit hacky...I think I only use it for testing to check that an object is definitely nanoarrow altrep (arrow does fancier things trying to roundtrip `arrow::ChunkedArray`s).
--
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
Re: [I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
Posted by "yutannihilation (via GitHub)" <gi...@apache.org>.
yutannihilation commented on issue #836:
URL: https://github.com/apache/arrow-nanoarrow/issues/836#issuecomment-3701061195
Quick update: new `R_altrep_class_name()` is a drop-in replacement for `CAR(ATTRIB(ALTREP_CLASS(x)))`.
https://github.com/r-devel/r-svn/commit/648c5b1786641af3e38a58c98cae08880968d323
--
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
Re: [I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
Posted by "yutannihilation (via GitHub)" <gi...@apache.org>.
yutannihilation commented on issue #836:
URL: https://github.com/apache/arrow-nanoarrow/issues/836#issuecomment-3695230421
This is the corresponding issue on arrow's repo.
https://github.com/apache/arrow/issues/48631
--
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
Re: [I] [R] `ATTRIB` will be removed [arrow-nanoarrow]
Posted by "paleolimbot (via GitHub)" <gi...@apache.org>.
paleolimbot closed issue #836: [R] `ATTRIB` will be removed
URL: https://github.com/apache/arrow-nanoarrow/issues/836
--
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: issues-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org