address issues

kernel/tests/cdf.rs

@@ -329,8 +329,9 @@ fn simple_cdf_version_ranges() -> DeltaResult<()> {
 #[test]
 fn update_operations() -> DeltaResult<()> {
     let batches = read_cdf_for_table("cdf-table-update-ops", 0, 2, None)?;
-    // Note: `update_pre` and `update_post` are technically not part of the delta spec, but are
-    // part of the tests used in delta
+    // Note: `update_pre` and `update_post` are technically not part of the delta spec, and instead
+    // should be `update_preimage` and `update_postimage` respectively. However, the tests in
+    // delta-spark use the post and pre.
     let mut expected = vec![
         "+----+--------------+-----------------+",
         "| id | _change_type | _commit_version |",
@@ -365,8 +366,6 @@ fn update_operations() -> DeltaResult<()> {
 #[test]
 fn false_data_change_is_ignored() -> DeltaResult<()> {
     let batches = read_cdf_for_table("cdf-table-data-change", 0, 1, None)?;
-    // Note: `update_pre` and `update_post` are technically not part of the delta spec, but are
-    // part of the tests used in delta
     let mut expected = vec![
         "+----+--------------+-----------------+",
         "| id | _change_type | _commit_version |",

kernel/tests/common/mod.rs

@@ -46,11 +46,9 @@ pub(crate) fn load_test_data(
     test_name: &str,
 ) -> Result<tempfile::TempDir, Box<dyn std::error::Error>> {
     let path = format!("{test_parent_dir}/{test_name}.tar.zst");
-    println!("Path: {path}");
     let tar = zstd::Decoder::new(std::fs::File::open(path)?)?;
     let mut archive = tar::Archive::new(tar);
     let temp_dir = tempfile::tempdir()?;
-    println!("dir : {:?}", temp_dir.path());
     archive.unpack(temp_dir.path())?;
     Ok(temp_dir)
 }