-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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-48823][DOCS] Improve clarity in lag
docstring
#47236
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. Could you please create a JIRA ticket under SPARK-44728?
@allisonwang-db Thanks! Sub ticket SPARK-48823 created. |
@thomhart31 thanks! please also add this JIRA in the PR title "[[SPARK-48823]..." |
lag
docstringlag
docstring
lag
docstringlag
docstring
Merged to master, Thank you @thomhart31 @allisonwang-db |
Sorry, I reverted this by 0fdebcc because the new format is incorrect. |
### What changes were proposed in this pull request? This PR edits grammar in `pyspark.sql.functions.lag` docstring. ### Why are the changes needed? To improve the documentation. ### Does this PR introduce any user-facing change? No changes in behavior are introduced. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#47236 from thomhart31/docs-lag. Authored-by: thomas.hart <thomas.hart@databricks.com> Signed-off-by: Kent Yao <yao@apache.org>
### What changes were proposed in this pull request? This PR edits grammar in `pyspark.sql.functions.lag` docstring. ### Why are the changes needed? To improve the documentation. ### Does this PR introduce any user-facing change? No changes in behavior are introduced. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#47236 from thomhart31/docs-lag. Authored-by: thomas.hart <thomas.hart@databricks.com> Signed-off-by: Kent Yao <yao@apache.org>
What is the current status of this PR, @thomhart31 and @yaooqinn ? Since we found that the content itself is invalid, do you want to update or shall we close this for now? |
Thank you, @dongjoon-hyun, for the reminder. I have committed a fix for the style issue. Hopefully, it can pass the GA. |
Merged to master. Thank you @thomhart31 @dongjoon-hyun @allisonwang-db |
### What changes were proposed in this pull request? This PR edits grammar in `pyspark.sql.functions.lag` docstring. ### Why are the changes needed? To improve the documentation. ### Does this PR introduce any user-facing change? No changes in behavior are introduced. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#47236 from thomhart31/docs-lag. Lead-authored-by: thomas.hart <thomas.hart@databricks.com> Co-authored-by: Kent Yao <yao@apache.org> Signed-off-by: Kent Yao <yao@apache.org>
### What changes were proposed in this pull request? This PR edits grammar in `pyspark.sql.functions.lag` docstring. ### Why are the changes needed? To improve the documentation. ### Does this PR introduce any user-facing change? No changes in behavior are introduced. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#47236 from thomhart31/docs-lag. Authored-by: thomas.hart <thomas.hart@databricks.com> Signed-off-by: Kent Yao <yao@apache.org>
### What changes were proposed in this pull request? This PR edits grammar in `pyspark.sql.functions.lag` docstring. ### Why are the changes needed? To improve the documentation. ### Does this PR introduce any user-facing change? No changes in behavior are introduced. ### How was this patch tested? Existing tests. ### Was this patch authored or co-authored using generative AI tooling? No Closes apache#47236 from thomhart31/docs-lag. Lead-authored-by: thomas.hart <thomas.hart@databricks.com> Co-authored-by: Kent Yao <yao@apache.org> Signed-off-by: Kent Yao <yao@apache.org>
What changes were proposed in this pull request?
This PR edits grammar in
pyspark.sql.functions.lag
docstring.Why are the changes needed?
To improve the documentation.
Does this PR introduce any user-facing change?
No changes in behavior are introduced.
How was this patch tested?
Existing tests.
Was this patch authored or co-authored using generative AI tooling?
No