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

[SPARK-2904] Remove non-used local variable in SparkSubmitArguments #1834

Closed
wants to merge 1 commit into from

Conversation

sarutak
Copy link
Member

@sarutak sarutak commented Aug 7, 2014

No description provided.

@SparkQA
Copy link

SparkQA commented Aug 7, 2014

QA tests have started for PR 1834. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18119/consoleFull

@SparkQA
Copy link

SparkQA commented Aug 7, 2014

QA results for PR 1834:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18119/consoleFull

@sryza
Copy link
Contributor

sryza commented Aug 7, 2014

LGTM

@pwendell
Copy link
Contributor

pwendell commented Aug 8, 2014

Yeah thanks - I can merge it.

asfgit pushed a commit that referenced this pull request Aug 8, 2014
Author: Kousuke Saruta <[email protected]>

Closes #1834 from sarutak/SPARK-2904 and squashes the following commits:

38e7d45 [Kousuke Saruta] Removed non-used variable in SparkSubmitArguments
(cherry picked from commit 9de6a42)

Signed-off-by: Patrick Wendell <[email protected]>
@asfgit asfgit closed this in 9de6a42 Aug 8, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: Kousuke Saruta <[email protected]>

Closes apache#1834 from sarutak/SPARK-2904 and squashes the following commits:

38e7d45 [Kousuke Saruta] Removed non-used variable in SparkSubmitArguments
@sarutak sarutak deleted the SPARK-2904 branch April 11, 2015 05:22
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.

4 participants