Error in return of __repr__

Author: timsaucer

python/datafusion/catalog.py

@@ -36,7 +36,7 @@ def __init__(self, catalog: df_internal.Catalog) -> None:
 
     def __repr__(self) -> str:
         """Print a string representation of the catalog."""
-        self.catalog.__repr__()
+        return self.catalog.__repr__()
 
     def names(self) -> list[str]:
         """Returns the list of databases in this catalog."""
@@ -56,7 +56,7 @@ def __init__(self, db: df_internal.Database) -> None:
 
     def __repr__(self) -> str:
         """Print a string representation of the database."""
-        self.db.__repr__()
+        return self.db.__repr__()
 
     def names(self) -> set[str]:
         """Returns the list of all tables in this database."""
@@ -76,7 +76,7 @@ def __init__(self, table: df_internal.Table) -> None:
 
     def __repr__(self) -> str:
         """Print a string representation of the table."""
-        self.table.__repr__()
+        return self.table.__repr__()
 
     @property
     def schema(self) -> pa.Schema:

python/datafusion/context.py

@@ -498,7 +498,7 @@ def __init__(
 
     def __repr__(self) -> str:
         """Print a string representation of the Session Context."""
-        self.ctx.__repr__()
+        return self.ctx.__repr__()
 
     @classmethod
     def global_ctx(cls) -> SessionContext:

python/datafusion/expr.py

@@ -1174,7 +1174,7 @@ def __init__(
 
     def __repr__(self) -> str:
         """Print a string representation of the window frame."""
-        self.window_frame.__repr__()
+        return self.window_frame.__repr__()
 
     def get_frame_units(self) -> str:
         """Returns the window frame units for the bounds."""

python/datafusion/user_defined.py

@@ -104,7 +104,7 @@ def __init__(
 
     def __repr__(self) -> str:
         """Print a string representation of the Scalar UDF."""
-        self._udf.__repr__()
+        return self._udf.__repr__()
 
     def __call__(self, *args: Expr) -> Expr:
         """Execute the UDF.
@@ -274,7 +274,7 @@ def __init__(
 
     def __repr__(self) -> str:
         """Print a string representation of the Aggregate UDF."""
-        self._udaf.__repr__()
+        return self._udaf.__repr__()
 
     def __call__(self, *args: Expr) -> Expr:
         """Execute the UDAF.
@@ -614,7 +614,7 @@ def __init__(
 
     def __repr__(self) -> str:
         """Print a string representation of the Window UDF."""
-        self._udwf.__repr__()
+        return self._udwf.__repr__()
 
     def __call__(self, *args: Expr) -> Expr:
         """Execute the UDWF.