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