Skip to content

Conversation

@allisonwang-db
Copy link
Contributor

What changes were proposed in this pull request?

This PR is a follow up for #52412. The location of the generated llms.txt file will be placed under the root of the Spark documentation: https://spark.apache.org/docs/llms.txt and links to the latest Spark documentations. We will add more versionless.md files for API specific docs.

Why are the changes needed?

To update the docs generation

Does this PR introduce any user-facing change?

No

How was this patch tested?

Tested locally

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

No

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

-1 because this seems to remove RDD Programing Guide accidentally.

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

If you want to remove rdd-programming-guide.html, please file a new JIRA issue instead of this follow-up, @allisonwang-db . The PR title should represent what the code change aims to do. There should be no hidden change.

@allisonwang-db
Copy link
Contributor Author

@dongjoon-hyun Sounds good I will create a separate PR. The intention here is that Spark Connect does not support the RDD API.

--docs-path . \
--output "$SELF/_site/llms.txt" \
--version "$SPARK_VERSION"
--output "$SELF/llms.txt" \
Copy link
Contributor

@cloud-fan cloud-fan Nov 13, 2025

Choose a reason for hiding this comment

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

Does mean we should only do it when releasing the latest major/feature version? We should'n overwrite it with an old version when releasing 4.0.1 for example.

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.

4 participants