Kurtiscwright opened a new issue, #2393: URL: https://github.com/apache/iceberg-rust/issues/2393
### Apache Iceberg Rust version None ### Describe the bug While running `make unit-test` for verifying the RC the update_table test in `/src/memory/catalog.rs` was failing because it checks for the `table` and `updated_table` to have different updated timestamps down to the ms, but on a fast enough machine the test can update the `updated_table` in the same ms as the `table` causing the test to incorrectly fail. ### To Reproduce `make test` ### Expected behavior Current: ```rust assert!(table.metadata().last_updated_ms() < updated_table.metadata().last_updated_ms()); ``` Fixed: ```rust assert!(table.metadata().last_updated_ms() <= updated_table.metadata().last_updated_ms()); ``` ### Willingness to contribute I can contribute a fix for this bug independently -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
