Skip to content

[Spark-14685] [CORE] Document heritability of localProperties #12455

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

Closed
wants to merge 3 commits into from

Conversation

marcintustin
Copy link
Contributor

What changes were proposed in this pull request?

This updates the java-/scala- doc for setLocalProperty to document heritability of localProperties. This also adds tests for that behaviour.

How was this patch tested?

Tests pass. New tests were added.

@marcintustin
Copy link
Contributor Author

@rxin Documentation and tests PR. Enhancement PR to follow.

@marcintustin marcintustin changed the title [Spark-14685] Document heritability of localProperties [Spark-14685] [CORE] Document heritability of localProperties Apr 17, 2016
@marcintustin
Copy link
Contributor Author

@JoshRosen This may well be of interest. As mentioned above, my enhancement PR is pending (running tests right now).

@SparkQA
Copy link

SparkQA commented Apr 18, 2016

Test build #2801 has finished for PR 12455 at commit b96cde1.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@marcintustin
Copy link
Contributor Author

@marcintustin
Copy link
Contributor Author

@rxin Are you cool to go ahead with this before we resolve the deal on #12456?

@rxin
Copy link
Contributor

rxin commented Apr 21, 2016

Thanks - let's merge this first once Jenkins pass.

@SparkQA
Copy link

SparkQA commented Apr 21, 2016

Test build #2847 has finished for PR 12455 at commit b96cde1.

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

@marcintustin
Copy link
Contributor Author

@rxin Tests have passed, can we merge? This still seems low risk for Spark-2.0

@rxin
Copy link
Contributor

rxin commented May 3, 2016

Thanks - merging in master / branch-2.0.

@asfgit asfgit closed this in 8028f3a May 3, 2016
asfgit pushed a commit that referenced this pull request May 3, 2016
## What changes were proposed in this pull request?

This updates the java-/scala- doc for setLocalProperty to document heritability of localProperties. This also adds tests for that behaviour.

## How was this patch tested?

Tests pass. New tests were added.

Author: Marcin Tustin <marcin.tustin@gmail.com>

Closes #12455 from marcintustin/SPARK-14685.

(cherry picked from commit 8028f3a)
Signed-off-by: Reynold Xin <rxin@databricks.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants