slfan1989 commented on code in PR #1675:
URL: https://github.com/apache/iceberg-rust/pull/1675#discussion_r2361504147
##########
crates/iceberg/src/transaction/update_statistics.rs:
##########
@@ -171,4 +171,56 @@ mod tests {
Some(statistics_file_2)
);
}
+
+ #[test]
+ fn test_set_single_statistics() {
Review Comment:
Thank you for your question! In fact, `test_no_statistics_set`,
`test_set_single_statistics`, and `test_update_statistics` test different
scenarios.
- `test_no_statistics_set`: Verifies that statistics_to_set is empty when no
statistics are set or removed, ensuring the system handles this correctly when
no operations are performed.
- `test_set_single_statistics`: Verifies the setting of a single statistics
file, ensuring that the set_statistics operation works correctly.
- `test_update_statistics`: Tests the setting and removal of multiple
statistics files, verifying the correct behavior of multiple set_statistics and
remove_statistics operations combined.
--
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]
