(iceberg-rust) branch main updated: docs(arrow): clarify ScanMetrics::bytes_read includes delete files (#2379)

[li...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

blackmwk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-rust.git


The following commit(s) were added to refs/heads/main by this push:
     new 0d97497bb docs(arrow): clarify ScanMetrics::bytes_read includes delete files (#2379)
0d97497bb is described below

commit 0d97497bb9dd46a6e3c3304e7c3855384b0119f2
Author: Matt Butrovich <mb...@users.noreply.github.com>
AuthorDate: Tue Apr 28 21:24:14 2026 -0400

    docs(arrow): clarify ScanMetrics::bytes_read includes delete files (#2379)
    
    ## Which issue does this PR close?
    
    - Closes #.
    
    ## What changes are included in this PR?
    
    Fix the `ScanMetrics::bytes_read` docstring. The counter is shared with
    `CachingDeleteFileLoader`, so it already includes delete-file reads, not
    just data files. Docstring now reflects that.
    
    ## Are these changes tested?
    
    Docstring-only change, no behavior change, so no new tests.
---
 crates/iceberg/src/arrow/scan_metrics.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/crates/iceberg/src/arrow/scan_metrics.rs b/crates/iceberg/src/arrow/scan_metrics.rs
index 4331a53fc..642190c57 100644
--- a/crates/iceberg/src/arrow/scan_metrics.rs
+++ b/crates/iceberg/src/arrow/scan_metrics.rs
@@ -66,7 +66,7 @@ impl ScanMetrics {
         &self.bytes_read
     }
 
-    /// Total bytes read from storage for data files during this scan.
+    /// Total bytes read from storage during this scan, including data files and delete files.
     pub fn bytes_read(&self) -> u64 {
         self.bytes_read.load(Ordering::Relaxed)
     }