[GitHub] [arrow] romainfrancois edited a comment on pull request #9615: ARROW-3316: [R] Multi-threaded conversion from R data.frame to Arrow table / record batch

[GitBox <gi...@apache.org>]

romainfrancois edited a comment on pull request #9615:
URL: https://github.com/apache/arrow/pull/9615#issuecomment-845787017


   Done. This probably won't have much impact, because I guess by the time the converter does `->ToArray()` there isn't much more work to be done because this is just a call to the builder's finish method: 
   
   ```cpp
   virtual Result<std::shared_ptr<Array>> ToArray() { return builder_->Finish(); }
   ```
   
   `Table__from_dots()` will need to be re-adapted again when we do chuncking. I think that each chunk will need its own Converter. 


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