You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/07 22:56:07 UTC

[GitHub] [arrow] kszucs commented on pull request #8650: ARROW-10530: [R] Use Converter API to convert SEXP to Array/ChunkedArray

kszucs commented on pull request #8650:
URL: https://github.com/apache/arrow/pull/8650#issuecomment-740230851


   > It all feels a bit weird to operate in terms of single values, i.e. via the `Append()` methods, since R does not have such concept, this has to resort to many casting.
   
   This converter API was designed to operate on list-like collections of generic objects one-by-one. We have different code paths for more efficient conversion from numpy/pandas, see [here](https://github.com/apache/arrow/blob/master/cpp/src/arrow/python/numpy_to_arrow.cc).
   
   > Would it make sense to have some sort of vectorised approach ?
   
   I think so, yes. We could extend the converter API to consume columns instead of individual values. 
   
   


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

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