Skip to content

Commit bc86d10

Browse files
authored
refactor(arrow): Rename parameter in delete_filter for clarity (#1955)
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.
1 parent 152aeb6 commit bc86d10

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

crates/iceberg/src/arrow/delete_filter.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,12 +82,12 @@ impl DeleteFilter {
8282
/// Retrieve a delete vector for a data file
8383
pub(crate) fn get_delete_vector_for_path(
8484
&self,
85-
delete_file_path: &str,
85+
data_file_path: &str,
8686
) -> Option<Arc<Mutex<DeleteVector>>> {
8787
self.state
8888
.read()
8989
.ok()
90-
.and_then(|st| st.delete_vectors.get(delete_file_path).cloned())
90+
.and_then(|st| st.delete_vectors.get(data_file_path).cloned())
9191
}
9292

9393
pub(crate) fn try_start_eq_del_load(&self, file_path: &str) -> Option<Arc<Notify>> {

0 commit comments

Comments
 (0)