Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-41649][CONNECT] Deduplicate docstrings in pyspark.sql.connect.window #39180

Closed
wants to merge 1 commit into from

Conversation

HyukjinKwon
Copy link
Member

What changes were proposed in this pull request?

This PR proposes to deduplicate docstrings in pyspark.sql.connect.window.

Why are the changes needed?

For easier maintenance

Does this PR introduce any user-facing change?

No, dev-only. There're mintor doc changes that mention about remote support in Apache Spark 3.4.

How was this patch tested?

No test. It has to be manually verified until we resolve SPARK-41653.

@HyukjinKwon
Copy link
Member Author

Merged to master.

rangeBetween.__doc__ = PySparkWindow.rangeBetween.__doc__


Window.__doc__ = Window.__doc__
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@HyukjinKwon is this a typo?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes :-). Mind creating a followup?

HyukjinKwon pushed a commit that referenced this pull request Dec 27, 2022
…l.connect.window

### What changes were proposed in this pull request?
This PR is a followup of #39180 that addresses a typo.

`Window.__doc__ = Window.__doc__`

### Why are the changes needed?
For easier maintenance

### Does this PR introduce _any_ user-facing change?
No.

### How was this patch tested?
Pass GA

Closes #39238 from bjornjorgensen/window-doc-PySparkWindowSpec.

Authored-by: Bjørn Jørgensen <47577197+bjornjorgensen@users.noreply.github.com>
Signed-off-by: Hyukjin Kwon <gurwls223@apache.org>
@HyukjinKwon HyukjinKwon deleted the SPARK-41649 branch January 15, 2024 00:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants