Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Mar 12, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Mar 12, 2015

Test build #28518 has started for PR 5002 at commit ced65f7.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

Is the fix the $@? LGTM.

@vanzin
Copy link
Contributor Author

vanzin commented Mar 12, 2015

Yeah. I also formatted things to look more like the other, similar code. Thanks to @nightwolfzor for pointing it out.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

CC @pwendell in case this has implications for the 1.3 release. Obviously looks OK to merge to master (jenkins won't test this anyway) but might even be needed at the last moment for 1.3.

@vanzin
Copy link
Contributor Author

vanzin commented Mar 12, 2015

No, this doesn't affect 1.3.

@srowen
Copy link
Member

srowen commented Mar 12, 2015

Ah right, this was just introduced with SPARK-4924. OK good to go

@SparkQA
Copy link

SparkQA commented Mar 12, 2015

Test build #28518 has finished for PR 5002 at commit ced65f7.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28518/
Test PASSed.

@asfgit asfgit closed this in 8f1bc79 Mar 12, 2015
@vanzin vanzin deleted the mkdist-hotfix branch March 20, 2015 23:34
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