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-32183][DOCS][PYTHON] User Guide - PySpark Usage Guide for Pandas with Apache Arrow #29548

Closed
wants to merge 3 commits into from

Conversation

HyukjinKwon
Copy link
Member

What changes were proposed in this pull request?

This PR proposes to move Arrow usage guide from Spark documentation site to PySpark documentation site (at "User Guide").

Here is the demo for reviewing quicker: https://hyukjin-spark.readthedocs.io/en/stable/user_guide/arrow_pandas.html

Why are the changes needed?

To have a single place for PySpark users, and better documentation.

Does this PR introduce any user-facing change?

Yes, it will move https://spark.apache.org/docs/latest/sql-pyspark-pandas-with-arrow.html to our PySpark documentation.

How was this patch tested?

cd docs
SKIP_SCALADOC=1 SKIP_RDOC=1 SKIP_SQLDOC=1 jekyll serve --watch

and

cd python/docs
make clean html

@HyukjinKwon
Copy link
Member Author

cc @rohitmishr1484 and @fhoering FYI

@SparkQA

This comment has been minimized.

@SparkQA
Copy link

SparkQA commented Aug 26, 2020

Test build #127925 has finished for PR 29548 at commit 268001e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Member

@viirya viirya left a comment

Choose a reason for hiding this comment

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

Few typos are found. Otherwise looks good to me. Thanks.

@HyukjinKwon
Copy link
Member Author

Thank you @viirya for reviewing this!

@HyukjinKwon
Copy link
Member Author

GitHub Actions builds passed. Merged to master.

@SparkQA
Copy link

SparkQA commented Aug 28, 2020

Test build #127979 has finished for PR 29548 at commit 56af102.

  • This patch fails due to an unknown error code, -9.
  • This patch merges cleanly.
  • This patch adds no public classes.

@BryanCutler
Copy link
Member

Sorry for the late review.. Looks great though, thanks!

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