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 23:24:03 UTC

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

kszucs edited a comment on pull request #8650:
URL: https://github.com/apache/arrow/pull/8650#issuecomment-740237766


   > Row-based conversion might make sense when converting an R list-type object, but otherwise you'd expect something vectorized to be better. For better or worse, now that all of this row-based conversion has been implemented here, we can benchmark both ways and compare.
   
   Sorry, it's not about row- vs. column-wise. Currently the converters have an `Append` method to convert and append values one-by-one. We can provide an `AppendMultiple` or something similar to extend the underlying arrays in batches so we can optimize for R's representation. 
   
   I'm going to take a look at the PR to properly answer Romain's questions.


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