(iceberg-python) branch main updated: fix: Remove unused fields from FileScanTask (#2777)

[fokko]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new 59dc8d13 fix: Remove unused fields from FileScanTask (#2777)
59dc8d13 is described below

commit 59dc8d13ad4e1500fff12946f1bfaddb5484f90e
Author: Drew Gallardo <img...@gmail.com>
AuthorDate: Fri Nov 21 06:45:06 2025 -0800

    fix: Remove unused fields from FileScanTask (#2777)
    
    Closes #2776
    
    # Rationale for this change
    
    This PR removes the unused `start` and `length` fields from the
    `FileScanTask`. While working on the models for rest scanning in #2775
    we noticed that these fields were initialized but never accessed or used
    anywhere in the code base.
    
    ## Are these changes tested?
    ```
    > make test
    ...
    3023 passed, 1392 deselected in 30.39s
    ```
    
    ## Are there any user-facing changes?
    no
---
 pyiceberg/table/__init__.py | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/pyiceberg/table/__init__.py b/pyiceberg/table/__init__.py
index 027992fb..2e26a4cc 100644
--- a/pyiceberg/table/__init__.py
+++ b/pyiceberg/table/__init__.py
@@ -1786,22 +1786,16 @@ class FileScanTask(ScanTask):
 
     file: DataFile
     delete_files: set[DataFile]
-    start: int
-    length: int
     residual: BooleanExpression
 
     def __init__(
         self,
         data_file: DataFile,
         delete_files: set[DataFile] | None = None,
-        start: int | None = None,
-        length: int | None = None,
         residual: BooleanExpression = ALWAYS_TRUE,
     ) -> None:
         self.file = data_file
         self.delete_files = delete_files or set()
-        self.start = start or 0
-        self.length = length or data_file.file_size_in_bytes
         self.residual = residual