Skip to content

Commit

Permalink
[SPARK-41757][CONNECT][PYTHON][FOLLOW-UP] Enable connect.functions.co…
Browse files Browse the repository at this point in the history
…l 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 #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 <sandeep@techaddict.me>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
(cherry picked from commit 2f49c1f)
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
  • Loading branch information
techaddict authored and HyukjinKwon committed Jan 27, 2023
1 parent d47974c commit 985efd7
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions python/pyspark/sql/connect/functions.py
Expand Up @@ -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__

Expand Down

0 comments on commit 985efd7

Please sign in to comment.