[GitHub] [arrow-rs] alamb commented on pull request #4393: Add Scalar/Datum abstraction (#1047)

["alamb (via GitHub)" <gi...@apache.org>]

alamb commented on PR #4393:
URL: https://github.com/apache/arrow-rs/pull/4393#issuecomment-1591185696

   At the very least the "Datum" notion allows for the inclusion of a Selection Vector at a later time -- perhaps like
   
   ```rust
   pub trait Datum {
   ...
     /// If there are certain rows that should be ignored by any kernels
     /// Defaults to None (all rows)
     fn selection(&self) -> Option<&BoolBuffer> { None }
   }
   ```


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

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