Skip to content

Commit 9ddeebc

Browse files
committed
bugfix + flushing on to_reader
1 parent b38eaf6 commit 9ddeebc

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

src/valor_lite/common/ephemeral.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,7 @@ def __exit__(self, exc_type, exc_val, exc_tb):
192192

193193
def to_reader(self) -> MemoryCacheReader:
194194
"""Get cache reader."""
195+
self.flush()
195196
return MemoryCacheReader(
196197
table=self._table, batch_size=self._batch_size
197198
)

src/valor_lite/common/persistent.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,8 @@ def iterate_tables(
169169
schema=self._schema,
170170
format="parquet",
171171
)
172-
for fragment in dataset.get_fragments(filter=filter):
173-
yield fragment.to_table(columns=columns)
172+
for fragment in dataset.get_fragments():
173+
yield fragment.to_table(columns=columns, filter=filter)
174174

175175
def iterate_fragments(self):
176176
"""Iterate over fragments within the file-based cache."""
@@ -425,4 +425,5 @@ def __exit__(self, exc_type, exc_val, exc_tb):
425425

426426
def to_reader(self) -> FileCacheReader:
427427
"""Get cache reader."""
428+
self.flush()
428429
return FileCacheReader.load(path=self.path)

0 commit comments

Comments
 (0)