Skip to content

[MINOR][DOCS] Typo fix in cloud-integration.md#34129

Closed
fishmandev wants to merge 1 commit intoapache:masterfrom
fishmandev:patch-1
Closed

[MINOR][DOCS] Typo fix in cloud-integration.md#34129
fishmandev wants to merge 1 commit intoapache:masterfrom
fishmandev:patch-1

Conversation

@fishmandev
Copy link
Contributor

What changes were proposed in this pull request?

Typo fix

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@github-actions github-actions bot added the DOCS label Sep 28, 2021
@HyukjinKwon HyukjinKwon changed the title Typo fix in cloud-integration.md [MINOR][DOCS] Typo fix in cloud-integration.md Sep 29, 2021
@HyukjinKwon
Copy link
Member

@fishmandev, let's make sure filling PR description although this is trivial change. While we're here, mind checking if there are more typos to fix around here?

Also, Github Actions in your forked repository should be enable to run CI, see also https://github.com/apache/spark/pull/34129/checks?check_run_id=3733907992

@fishmandev
Copy link
Contributor Author

Also, Github Actions in your forked repository should be enable to run CI

Done it.

While we're here, mind checking if there are more typos to fix around here?

I have a question regarding this part of the text:
In order to store hundreds of petabytes of data without any single points of failure, object stores replace the classic file system directory tree with a simpler model of `object-name => data`.

Why is the plural used for 'SPOF' in this context? Or is it a typo?
... without any single point of failure, ...

@HyukjinKwon, what do you think about it?

@srowen
Copy link
Member

srowen commented Oct 1, 2021

Either works, to my ear. There is not one SPOF, there are not multiple SPOFs => there are none

@srowen
Copy link
Member

srowen commented Oct 1, 2021

Jenkins test this please

@SparkQA
Copy link

SparkQA commented Oct 1, 2021

Test build #143785 has finished for PR 34129 at commit 7e6ca38.

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

@SparkQA
Copy link

SparkQA commented Oct 1, 2021

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48297/

@SparkQA
Copy link

SparkQA commented Oct 1, 2021

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/48297/

@srowen srowen closed this in 25db6b4 Oct 1, 2021
@srowen
Copy link
Member

srowen commented Oct 1, 2021

Merged to master

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants