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-44746][PYTHON] Add more Python UDTF documentation for functions that accept input tables #45375

Closed
wants to merge 9 commits into from

Conversation

dtenedor
Copy link
Contributor

@dtenedor dtenedor commented Mar 4, 2024

What changes were proposed in this pull request?

This PR adds more Python UDTF documentation for functions that accept input tables.

Why are the changes needed?

This functionality was added recently but not covered in docs yet.

Does this PR introduce any user-facing change?

No, it's a documentation-only change.

How was this patch tested?

N/A

Was this patch authored or co-authored using generative AI tooling?

No

@dtenedor
Copy link
Contributor Author

dtenedor commented Mar 4, 2024

cc @allisonwang-db @ueshin

Copy link
Contributor

@allisonwang-db allisonwang-db left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for adding it.

python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
Copy link
Contributor Author

@dtenedor dtenedor left a comment

Choose a reason for hiding this comment

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

Thanks @allisonwang-db for your review! Responded to your comments.

python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
python/docs/source/user_guide/sql/python_udtf.rst Outdated Show resolved Hide resolved
@dtenedor dtenedor requested a review from allisonwang-db March 5, 2024 18:34
spark.sql("CREATE TABLE values_table (a STRING, b INT)")
spark.sql("INSERT INTO values_table VALUES ('abc', 2), ('abc', 4), ('def', 6), ('def', 8)")
spark.table("values_table").show()

Copy link
Contributor

Choose a reason for hiding this comment

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

maybe we can have the output of the show() here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, this is done.

@allisonwang-db
Copy link
Contributor

Looks good! Also cc @ueshin and @HyukjinKwon

@HyukjinKwon HyukjinKwon changed the title [SPARK-44746][Python] Add more Python UDTF documentation for functions that accept input tables [SPARK-44746][PYTHON] Add more Python UDTF documentation for functions that accept input tables Mar 6, 2024
@HyukjinKwon
Copy link
Member

Merged to master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants