deprecate Expr::display_name

Ref: apache/datafusion#11797

Author: Michael-J-Ward

python/datafusion/expr.py

@@ -29,6 +29,7 @@
 )
 from datafusion.common import NullTreatment, RexType, DataTypeMap
 from typing import Any, Optional, Type
+from typing_extensions import deprecated
 import pyarrow as pa
 
 # The following are imported from the internal representation. We may choose to
@@ -195,12 +196,20 @@ def to_variant(self) -> Any:
         """Convert this expression into a python object if possible."""
         return self.expr.to_variant()
 
+    @deprecated("display_name() is deprecated. Use :py:meth:`~Expr.schema_name` instead")
     def display_name(self) -> str:
         """Returns the name of this expression as it should appear in a schema.
 
         This name will not include any CAST expressions.
         """
-        return self.expr.display_name()
+        return self.schema_name()
+
+    def schema_name(self) -> str:
+        """Returns the name of this expression as it should appear in a schema.
+
+        This name will not include any CAST expressions.
+        """
+        return self.expr.schema_name()
 
     def canonical_name(self) -> str:
         """Returns a complete string representation of this expression."""

python/datafusion/tests/test_expr.py

@@ -192,3 +192,23 @@ def test_expr_getitem() -> None:
 
     assert names == ["Alice", "Bob", "Charlie", None]
     assert array_values == [2, 5, None, None]
+
+
+def test_display_name_deprecation():
+    import warnings
+    expr = col("foo")
+    with warnings.catch_warnings(record=True) as w:
+        # Cause all warnings to always be triggered
+        warnings.simplefilter("always")
+
+        # should trigger warning
+        name = expr.display_name()
+
+        # Verify some things
+        assert len(w) == 1
+        assert issubclass(w[-1].category, DeprecationWarning)
+        assert "deprecated" in str(w[-1].message)
+
+    # returns appropriate result
+    assert name == expr.schema_name()
+    assert name == "foo"

src/expr.rs

@@ -189,7 +189,7 @@ impl PyExpr {
 
     /// Returns the name of this expression as it should appear in a schema. This name
     /// will not include any CAST expressions.
-    fn display_name(&self) -> PyResult<String> {
+    fn schema_name(&self) -> PyResult<String> {
         Ok(format!("{}", self.expr.schema_name()))
     }