From 03f08d010d70b756d71f273bf999426e803f5a0e Mon Sep 17 00:00:00 2001 From: Robert Mu Date: Tue, 23 Dec 2025 10:26:32 +0800 Subject: [PATCH] refactor(arrow): Rename parameter in delete_filter for clarity The parameter `delete_file_path` in `get_delete_vector_for_path` was misleading because the function expects a data file path, not a delete file path. Renaming it to `data_file_path` accurately reflects its usage. --- crates/iceberg/src/arrow/delete_filter.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/iceberg/src/arrow/delete_filter.rs b/crates/iceberg/src/arrow/delete_filter.rs index e6434e72ec..4af9f6b6ff 100644 --- a/crates/iceberg/src/arrow/delete_filter.rs +++ b/crates/iceberg/src/arrow/delete_filter.rs @@ -82,12 +82,12 @@ impl DeleteFilter { /// Retrieve a delete vector for a data file pub(crate) fn get_delete_vector_for_path( &self, - delete_file_path: &str, + data_file_path: &str, ) -> Option>> { self.state .read() .ok() - .and_then(|st| st.delete_vectors.get(delete_file_path).cloned()) + .and_then(|st| st.delete_vectors.get(data_file_path).cloned()) } pub(crate) fn try_start_eq_del_load(&self, file_path: &str) -> Option> {