Skip to content

Commit 3b60138

Browse files
committed
fix
1 parent 7406b1e commit 3b60138

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

paimon-python/pypaimon/read/scanner/full_starting_scanner.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -82,15 +82,17 @@ def _read_manifest_files(self) -> List[ManifestFileMeta]:
8282
if not latest_snapshot:
8383
return []
8484
manifest_files = self.manifest_list_manager.read_all(latest_snapshot)
85+
return self.filter_manifest_files(manifest_files)
8586

86-
def test_predicate(file: ManifestFileMeta) -> bool:
87+
def filter_manifest_files(self, files: List[ManifestFileMeta]) -> List[ManifestFileMeta]:
88+
def filter_manifest_file(file: ManifestFileMeta) -> bool:
8789
if not self.partition_key_predicate:
8890
return True
8991
return self.partition_key_predicate.test_by_simple_stats(
9092
file.partition_stats,
9193
file.num_added_files + file.num_deleted_files)
9294

93-
return [file for file in manifest_files if test_predicate(file)]
95+
return [file for file in files if filter_manifest_file(file)]
9496

9597
def plan_files(self) -> List[ManifestEntry]:
9698
manifest_files = self._read_manifest_files()

paimon-python/pypaimon/read/scanner/incremental_starting_scanner.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,10 +43,12 @@ def plan_files(self) -> List[ManifestEntry]:
4343
for snapshot in snapshots_in_range:
4444
# Get manifest files for this snapshot
4545
manifest_files = self.manifest_list_manager.read_delta(snapshot)
46+
manifest_files = self.filter_manifest_files(manifest_files)
4647

4748
# Read all entries from manifest files
4849
for manifest_file in manifest_files:
49-
entries = self.manifest_file_manager.read(manifest_file.file_name)
50+
entries = self.manifest_file_manager.read(manifest_file.file_name,
51+
lambda row: self._filter_manifest_entry(row))
5052
file_entries.extend(entries)
5153
return file_entries
5254

0 commit comments

Comments
 (0)