-
Notifications
You must be signed in to change notification settings - Fork 28.3k
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
SPARK-1307 [DOCS] Don't use term 'standalone' to refer to a Spark Application #2787
Conversation
…usion with standalone deployment mode. And fix placement of reference to this in MLlib docs.
Test FAILed. |
I think this is a good idea. I'll leave this here so others can comment, but this LGTM as it is. |
test this please |
LGTM too. Waiting for Jenkins. |
Test FAILed. |
We probably don't need to run tests on this one since it modifies only docs. |
I tested locally but It is still safer to see Jenkins happy. |
test this please |
QA tests have started for PR 2787 at commit
|
QA tests have finished for PR 2787 at commit
|
Test PASSed. |
Merged into master. Thanks! |
HT to Diana, just proposing an implementation of her suggestion, which I rather agreed with. Is there a second/third for the motion?
Refer to "self-contained" rather than "standalone" apps to avoid confusion with standalone deployment mode. And fix placement of reference to this in MLlib docs.