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-47299][PYTHON][DOCS] Use the same versions.json in the dropdown of different versions of PySpark documents #45400

Closed
wants to merge 3 commits into from

Conversation

panbingkun
Copy link
Contributor

@panbingkun panbingkun commented Mar 6, 2024

What changes were proposed in this pull request?

The pr aims to use the same versions.json in the dropdown of different versions of PySpark documents.

Why are the changes needed?

As discussed in the email group, using this approach can avoid maintenance difficulties and inconsistencies that may arise when multi active release version lines are released in the future.
image

Does this PR introduce any user-facing change?

Yes, only for pyspark docs.

How was this patch tested?

  • Manually test.
  • Pass GA.

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

No.

…own of different versions of PySpark documents
@panbingkun panbingkun marked this pull request as ready for review March 6, 2024 10:47
@panbingkun
Copy link
Contributor Author

cc @HyukjinKwon @HeartSaVioR

Copy link
Contributor

@HeartSaVioR HeartSaVioR left a comment

Choose a reason for hiding this comment

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

+1

@panbingkun Could you please remove the versions.json in pyspark docs as well to be not confused?

@HeartSaVioR
Copy link
Contributor

(BTW sorry for visiting this too lately. Was very busy with other stuffs.)

@panbingkun
Copy link
Contributor Author

panbingkun commented Apr 8, 2024

+1

@panbingkun Could you please remove the versions.json in pyspark docs as well to be not confused?

Done

@HeartSaVioR Thank you very much for taking the time to review this PR. ❤️

@HeartSaVioR
Copy link
Contributor

Thanks! Merging to master/3.5.

HeartSaVioR pushed a commit that referenced this pull request Apr 8, 2024
…own of different versions of PySpark documents

### What changes were proposed in this pull request?
The pr aims to use the same `versions.json` in the dropdown of `different versions` of PySpark documents.

### Why are the changes needed?
As discussed in the email group, using this approach can avoid `maintenance difficulties` and `inconsistencies` that may arise when `multi active release version lines` are released in the future.
<img width="798" alt="image" src="https://github.com/apache/spark/assets/15246973/8a08a4fe-e1fb-4334-a3f9-c6dffb01cbd6">

### Does this PR introduce _any_ user-facing change?
Yes, only for pyspark docs.

### How was this patch tested?
- Manually test.
- Pass GA.

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

Closes #45400 from panbingkun/SPARK-47299.

Authored-by: panbingkun <panbingkun@baidu.com>
Signed-off-by: Jungtaek Lim <kabhwan.opensource@gmail.com>
(cherry picked from commit b299b2b)
Signed-off-by: Jungtaek Lim <kabhwan.opensource@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants