From 985efd78e5ee9937efcb78d79c5e2634789bf9c7 Mon Sep 17 00:00:00 2001 From: Sandeep Singh Date: Fri, 27 Jan 2023 11:31:15 +0900 Subject: [PATCH] [SPARK-41757][CONNECT][PYTHON][FOLLOW-UP] Enable connect.functions.col doctest ### What changes were proposed in this pull request? To reenable the doc tests for `col` Function, this patch makes the string representation of the Column closer to the regular PySpark Column. This PR is a follow up to https://github.com/apache/spark/pull/39616 with enabling col doctests ### Why are the changes needed? Improve Coverage ### Does this PR introduce any user-facing change? No ### How was this patch tested? Reenabled doc tests. Closes #39761 from techaddict/SPARK-41757-2. Authored-by: Sandeep Singh Signed-off-by: Hyukjin Kwon (cherry picked from commit 2f49c1fed0e9962b2b29ea9017edccbd52a5ce8e) Signed-off-by: Hyukjin Kwon --- python/pyspark/sql/connect/functions.py | 3 --- 1 file changed, 3 deletions(-) diff --git a/python/pyspark/sql/connect/functions.py b/python/pyspark/sql/connect/functions.py index ee7b45622b3e1..7c21f9280c2ca 100644 --- a/python/pyspark/sql/connect/functions.py +++ b/python/pyspark/sql/connect/functions.py @@ -2453,9 +2453,6 @@ def _test() -> None: del pyspark.sql.connect.functions.timestamp_seconds.__doc__ del pyspark.sql.connect.functions.unix_timestamp.__doc__ - # TODO(SPARK-41757): Fix String representation for Column class - del pyspark.sql.connect.functions.col.__doc__ - # TODO(SPARK-41812): Proper column names after join del pyspark.sql.connect.functions.count_distinct.__doc__