alamb opened a new pull request, #8022:
URL: https://github.com/apache/arrow-rs/pull/8022

   # Which issue does this PR close?
   
   - Part of XXX
   
   
   
   
   # Rationale for this change
   
   Right now the InMemoryRowGroup does both the IO and the CPU work to on a 
call to `fetch`
   
   in XXX I am trying to separate the two, so that we can have more control over
   when IO happens and when CPU work happens. I figured I could refactor the
   InMemoryRowGroup as a separate PR to make it easier to review and convince 
myself this refactor is correct.
   
   # What changes are included in this PR?
   
   Refactor `InMemoryRowGroup::fetch` into three functions
   
   # Are these changes tested?
   
   By existing CI
   
   If tests are not included in your PR, please explain why (for example, are 
they covered by existing tests)?
   
   # Are there any user-facing changes?
   
   No


-- 
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-unsubscr...@arrow.apache.org

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

Reply via email to