Skip to content

Commit e494bed

Browse files
timsaucerclaude
andcommitted
Remove redundant read_empty Rust binding, make Python read_empty an alias for empty_table
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 30fc3d5 commit e494bed

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

crates/core/src/context.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1217,11 +1217,6 @@ impl PySessionContext {
12171217
Ok(PyDataFrame::new(df))
12181218
}
12191219

1220-
pub fn read_empty(&self) -> PyDataFusionResult<PyDataFrame> {
1221-
let df = self.ctx.read_empty()?;
1222-
Ok(PyDataFrame::new(df))
1223-
}
1224-
12251220
#[pyo3(signature = (path, schema=None, file_extension=".arrow", table_partition_cols=vec![]))]
12261221
pub fn read_arrow(
12271222
&self,

python/datafusion/context.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1392,10 +1392,12 @@ def read_arrow(
13921392
def read_empty(self) -> DataFrame:
13931393
"""Create an empty :py:class:`DataFrame` with no columns or rows.
13941394
1395+
This is an alias for :meth:`empty_table`.
1396+
13951397
Returns:
13961398
An empty DataFrame.
13971399
"""
1398-
return DataFrame(self.ctx.read_empty())
1400+
return self.empty_table()
13991401

14001402
def read_table(
14011403
self, table: Table | TableProviderExportable | DataFrame | pa.dataset.Dataset

0 commit comments

Comments
 (0)