Skip to content

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 <[email protected]>

Closes #12455 from marcintustin/SPARK-14685.

(cherry picked from commit 8028f3a)
Signed-off-by: Reynold Xin <[email protected]>
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