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

[MINOR] [DOC] Document local[*,F] master modes #16562

Closed
wants to merge 1 commit into from

Conversation

mcuelenaere
Copy link
Contributor

What changes were proposed in this pull request?

core/src/main/scala/org/apache/spark/SparkContext.scala contains LOCAL_N_FAILURES_REGEX master mode, but this was never documented, so do so.

How was this patch tested?

By using the Github Markdown preview feature.

@srowen
Copy link
Member

srowen commented Jan 12, 2017

This functionality is so obscure I'd suspect it should be deprecated. Does anyone have a thought about that?

In any event, it's fair enough to document it.

@srowen
Copy link
Member

srowen commented Jan 14, 2017

@mcuelenaere per http://spark.apache.org/contributing.html would you add [MINOR][DOC] to the title?

@SparkQA
Copy link

SparkQA commented Jan 14, 2017

Test build #3532 has finished for PR 16562 at commit 6e548a5.

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

@mcuelenaere mcuelenaere changed the title Document local[*,F] master modes [MINOR] [DOC] Document local[*,F] master modes Jan 14, 2017
core/src/main/scala/org/apache/spark/SparkContext.scala contains LOCAL_N_FAILURES_REGEX master mode, but this was never documented, so do so.
@mcuelenaere
Copy link
Contributor Author

@srowen done

@srowen
Copy link
Member

srowen commented Jan 15, 2017

Merged to master

@asfgit asfgit closed this in 3df2d93 Jan 15, 2017
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
## What changes were proposed in this pull request?

core/src/main/scala/org/apache/spark/SparkContext.scala contains LOCAL_N_FAILURES_REGEX master mode, but this was never documented, so do so.

## How was this patch tested?

By using the Github Markdown preview feature.

Author: Maurus Cuelenaere <mcuelenaere@gmail.com>

Closes apache#16562 from mcuelenaere/patch-1.
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
## What changes were proposed in this pull request?

core/src/main/scala/org/apache/spark/SparkContext.scala contains LOCAL_N_FAILURES_REGEX master mode, but this was never documented, so do so.

## How was this patch tested?

By using the Github Markdown preview feature.

Author: Maurus Cuelenaere <mcuelenaere@gmail.com>

Closes apache#16562 from mcuelenaere/patch-1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants