Skip to content

Commit 2cd3d94

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 e421f7d commit 2cd3d94

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 {
@@ -1150,7 +1145,6 @@ pub(crate) fn init_module(m: &Bound<'_, PyModule>) -> PyResult<()> {
11501145
m.add_wrapped(wrap_pyfunction!(map_values))?;
11511146
m.add_wrapped(wrap_pyfunction!(map_extract))?;
11521147
m.add_wrapped(wrap_pyfunction!(map_entries))?;
1153-
m.add_wrapped(wrap_pyfunction!(element_at))?;
11541148

11551149
// Window Functions
11561150
m.add_wrapped(wrap_pyfunction!(lead))?;

0 commit comments

Comments
 (0)