Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-12-14 Thread via GitHub
sungwy closed issue #706: Bug in `PartialEq` for `Struct` URL: https://github.com/apache/iceberg-rust/issues/706 -- 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,

Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-12-14 Thread via GitHub
sungwy commented on issue #706: URL: https://github.com/apache/iceberg-rust/issues/706#issuecomment-2543139629 Closing this issue -- 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 comm

Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-12-10 Thread via GitHub
sungwy commented on issue #706: URL: https://github.com/apache/iceberg-rust/issues/706#issuecomment-2531897325 Hi @Sl1mb0 - Thank you for raising this issue! I took a look at this issue in order to resolve all issues blocking the 0.4.0 release. I put up the PR #772 to replicate your i

Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-11-20 Thread via GitHub
Sl1mb0 commented on issue #706: URL: https://github.com/apache/iceberg-rust/issues/706#issuecomment-2488967779 Sure! ```rust mod tests { use std::collections::HashMap; use std::io::Seek; use std::sync::Arc; use arrow::{ array::Float16Buffer

Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-11-20 Thread via GitHub
liurenjie1024 commented on issue #706: URL: https://github.com/apache/iceberg-rust/issues/706#issuecomment-2488664891 Thanks for reporting this, could you provide a more concrete example to reproduce this? -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [I] Bug in `PartialEq` for `Struct` [iceberg-rust]

2024-11-20 Thread via GitHub
alamb commented on issue #706: URL: https://github.com/apache/iceberg-rust/issues/706#issuecomment-2488343199 In case it helps, here is the workaround we used: (there is something about the `partition` on `DataFile` that doesn't compare as equal ```rust /// Validates tha