Skip to content

Commit 4827528

Browse files
timsaucerclaude
andcommitted
Remove redundant pyo3 element_at function
element_at is already a Python-only alias for map_extract, so the Rust binding is unnecessary. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent cce1305 commit 4827528

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

crates/core/src/functions.rs

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -100,11 +100,6 @@ fn make_map(keys: Vec<PyExpr>, values: Vec<PyExpr>) -> PyExpr {
100100
datafusion::functions_nested::map::map(keys, values).into()
101101
}
102102

103-
#[pyfunction]
104-
fn element_at(map: PyExpr, key: PyExpr) -> PyExpr {
105-
datafusion::functions_nested::expr_fn::map_extract(map.into(), key.into()).into()
106-
}
107-
108103
#[pyfunction]
109104
#[pyo3(signature = (array, element, index=None))]
110105
fn array_position(array: PyExpr, element: PyExpr, index: Option<i64>) -> PyExpr {
@@ -1148,7 +1143,6 @@ pub(crate) fn init_module(m: &Bound<'_, PyModule>) -> PyResult<()> {
11481143
m.add_wrapped(wrap_pyfunction!(map_values))?;
11491144
m.add_wrapped(wrap_pyfunction!(map_extract))?;
11501145
m.add_wrapped(wrap_pyfunction!(map_entries))?;
1151-
m.add_wrapped(wrap_pyfunction!(element_at))?;
11521146

11531147
// Window Functions
11541148
m.add_wrapped(wrap_pyfunction!(lead))?;

0 commit comments

Comments
 (0)