alamb opened a new issue, #7684:
URL: https://github.com/apache/arrow-rs/issues/7684

   Yeah, I think in general this is hitting the tension you mentioned in the 
earlier PRs of early vs late validation.
   
   The more I think about it the more I think we should move the validation to 
construction because:
   1. The only reason to create a `Variant` in the first place is to access its 
data, so I think we would end up validating it almost immediately on read
   2. `Variants` are constructed once but read many times so validating up 
front is probably faster
   3. We could offer an `unchecked` variant for construction if performance 
overhead is hight that skips the validation
   
   _Originally posted by @alamb in 
https://github.com/apache/arrow-rs/pull/7666#discussion_r2151997857_
               


-- 
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.apache.org

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

Reply via email to