-
Notifications
You must be signed in to change notification settings - Fork 28.6k
[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
Conversation
@rxin Documentation and tests PR. Enhancement PR to follow. |
@JoshRosen This may well be of interest. As mentioned above, my enhancement PR is pending (running tests right now). |
Test build #2801 has finished for PR 12455 at commit
|
The test failures are probably bogus, as per http://mail-archives.apache.org/mod_mbox/spark-dev/201604.mbox/%3CCAMFhwAYRbN0yJGwzvrY8atzS9CCudzioF%3DbcGogCwPq3gPC6Uw%40mail.gmail.com%3E |
Thanks - let's merge this first once Jenkins pass. |
Test build #2847 has finished for PR 12455 at commit
|
Thanks - merging in master / branch-2.0. |
## 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>
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.