romainfrancois commented on pull request #9615:
URL: https://github.com/apache/arrow/pull/9615#issuecomment-845206073


   After all the `DelayedExtend()` are finished, i.e. after the `status &= 
tasks.Finish();` thing, there is still work to do to actually create the 
arrays: 
   
   ```cpp
   for (int j = 0; j < num_fields; j++) {
       auto& converter = converters[j];
       if (converter != nullptr) {
         auto maybe_array = converter->ToArray();
         StopIfNotOk(maybe_array.status());
         columns[j] = 
std::make_shared<arrow::ChunkedArray>(maybe_array.ValueUnsafe());
       }
     }
   ```
   
   I don't think there's any R involved there, so I suppose this could be done 
in parallel, with some care about the `StopIfNotOk()` ...


-- 
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:
[email protected]


Reply via email to