You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2025/07/17 15:40:05 UTC
[I] `validated` and `is_fully_validated` flags doesn't need to be part of PartialEq [arrow-rs]
alamb opened a new issue, #7952:
URL: https://github.com/apache/arrow-rs/issues/7952
It seems like the `validated` and `is_fully_validated` flags doesn't need to be part of a logical type check? Like two variants can be equal by value even if one is fully validated and one is not
I would expect the following to pass for all variants and metadata
```rust
let variant1 = Variant::new(metadata, buffers);
let variant2 = Variant::new(metadata, buffers).with_full_validation();
assert_eq!(variant1, variant2)
```
_Originally posted by @alamb in https://github.com/apache/arrow-rs/pull/7943#discussion_r2213119799_
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] `validated` and `is_fully_validated` flags doesn't need to be part of PartialEq [arrow-rs]
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on issue #7952:
URL: https://github.com/apache/arrow-rs/issues/7952#issuecomment-3130014299
`label_issue.py` automatically added labels {'parquet'} from #7961
--
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
Re: [I] `validated` and `is_fully_validated` flags doesn't need to be part of PartialEq [arrow-rs]
Posted by "abacef (via GitHub)" <gi...@apache.org>.
abacef commented on issue #7952:
URL: https://github.com/apache/arrow-rs/issues/7952#issuecomment-3085243659
take
--
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
Re: [I] `validated` and `is_fully_validated` flags doesn't need to be part of PartialEq [arrow-rs]
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb closed issue #7952: `validated` and `is_fully_validated` flags doesn't need to be part of PartialEq
URL: https://github.com/apache/arrow-rs/issues/7952
--
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