Skip to content

[MINOR][DOCS] Fix various broken links and link anchors#45022

Closed
nchammas wants to merge 4 commits intoapache:masterfrom
nchammas:more-broken-links
Closed

[MINOR][DOCS] Fix various broken links and link anchors#45022
nchammas wants to merge 4 commits intoapache:masterfrom
nchammas:more-broken-links

Conversation

@nchammas
Copy link
Contributor

@nchammas nchammas commented Feb 4, 2024

What changes were proposed in this pull request?

Fix various broken links and link anchors.

Why are the changes needed?

Broken is broken, savvy?

Does this PR introduce any user-facing change?

Yes, it fixes broken links in user-facing documentation so they're no longer broken.

How was this patch tested?

I built the docs and clicked on some of the fixed links to confirm they are no longer broken.

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

No.

The `spark-submit` script can load default [Spark configuration values](configuration.html) from a
properties file and pass them on to your application. By default, it will read options
from `conf/spark-defaults.conf` in the Spark directory. For more detail, see the section on
[loading default configurations](configuration.html#loading-default-configurations).
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This section doesn't exist anymore. I couldn't find something similar to link to, so I just deleted this sentence.

**Configuration**

In order to enable this recovery mode, you can set SPARK_DAEMON_JAVA_OPTS in spark-env by configuring `spark.deploy.recoveryMode` and related spark.deploy.zookeeper.* configurations.
For more information about these configurations please refer to the [configuration doc](configuration.html#deploy)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The #deploy section doesn't exist anymore. Neither does documentation for the spark.deploy.* configs mentioned here.

@yaooqinn yaooqinn closed this in 07f1341 Feb 5, 2024
@yaooqinn
Copy link
Member

yaooqinn commented Feb 5, 2024

Thanks, merged to master

@nchammas nchammas deleted the more-broken-links branch February 5, 2024 06:39
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.

2 participants