Erigara commented on code in PR #1395:
URL: https://github.com/apache/iceberg-rust/pull/1395#discussion_r2132028126


##########
crates/integration_tests/tests/shared_tests/append_data_file_test.rs:
##########
@@ -132,7 +132,10 @@ async fn test_append_data_file() {
 
     // commit result again
     let tx = Transaction::new(&table);
-    let mut append_action = tx.fast_append(None, vec![]).unwrap();
+    let mut append_action = tx
+        .fast_append(None, vec![])
+        .unwrap()
+        .with_check_duplicate(false);

Review Comment:
   By just looking at the test idk why we are trying to commit the same files 
twice.
   So i agree let's remove it.



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to