nealrichardson commented on pull request #10445:
URL: https://github.com/apache/arrow/pull/10445#issuecomment-858109294


   > I've used `arrow.use_altrep` but I can use `arrow.altrep` all the same.
   
   SGTM
   
   > 
   > I've put `GetBoolOption("arrow.altrep", true)` after the 
`array->null_count() == 0` because it's cheaper to call `null_count()`.
   
   That surprises me, I'd expect that checking an R boolean would be faster 
than counting nulls in a (potentially long) array.
   
   > 
   > `GetBoolOption("arrow.altrep", true)` means that this is opt-out, I'm fine 
with this but it might be risky.
   
   I think defaulting to ALTREP on is good (particularly if/when we quantify 
its benefits) but leaving an escape hatch in case of error is nice.
   
   


-- 
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:
us...@infra.apache.org


Reply via email to