Skip to content

Conversation

@davies
Copy link
Contributor

@davies davies commented Feb 25, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Feb 25, 2015

Test build #27962 has started for PR 4772 at commit 389f0c6.

  • This patch merges cleanly.

@jkbradley
Copy link
Member

LGTM, worked on my machine

@SparkQA
Copy link

SparkQA commented Feb 26, 2015

Test build #27962 has finished for PR 4772 at commit 389f0c6.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link

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

@SparkQA
Copy link

SparkQA commented Feb 26, 2015

Test build #621 has started for PR 4772 at commit 389f0c6.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 26, 2015

Test build #621 has finished for PR 4772 at commit 389f0c6.

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

@rxin
Copy link
Contributor

rxin commented Feb 26, 2015

Merging in master. Thanks!

asfgit pushed a commit that referenced this pull request Feb 26, 2015
Author: Davies Liu <[email protected]>

Closes #4772 from davies/source_link and squashes the following commits:

389f0c6 [Davies Liu] fix link to source code in Pyton API docs
asfgit pushed a commit that referenced this pull request Feb 26, 2015
Author: Davies Liu <[email protected]>

Closes #4772 from davies/source_link and squashes the following commits:

389f0c6 [Davies Liu] fix link to source code in Pyton API docs

(cherry picked from commit 015895a)
Signed-off-by: Reynold Xin <[email protected]>
@jkbradley
Copy link
Member

@rxin Uh, this was meant to be a backport for branch-1.2. It was already fixed in master and branch-1.3 by [https://github.com//pull/4756]

@rxin
Copy link
Contributor

rxin commented Feb 26, 2015

Yup I meant 1.2... I accidentally also cherry picked it into 1.3, which is a noop.

@andrewor14
Copy link
Contributor

Hey @davies can you close this since it's already merged?

@davies davies closed this Feb 26, 2015
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.

6 participants