Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Feb 29, 2016

What changes were proposed in this pull request?

Just fixed the log place introduced by #11401

How was this patch tested?

unit tests.

@zsxwing
Copy link
Member Author

zsxwing commented Feb 29, 2016

cc @andrewor14 Just found I added the log in a wrong place :(

@andrewor14
Copy link
Contributor

Oops, LGTM

@asfgit asfgit closed this in 644dbb6 Feb 29, 2016
@zsxwing zsxwing deleted the SPARK-13522-follow-up branch February 29, 2016 20:54
@SparkQA
Copy link

SparkQA commented Feb 29, 2016

Test build #52195 has finished for PR 11432 at commit f5e8608.

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

roygao94 pushed a commit to roygao94/spark that referenced this pull request Mar 22, 2016
## What changes were proposed in this pull request?

Just fixed the log place introduced by apache#11401

## How was this patch tested?

unit tests.

Author: Shixiong Zhu <[email protected]>

Closes apache#11432 from zsxwing/SPARK-13522-follow-up.
asfgit pushed a commit that referenced this pull request May 11, 2016
## What changes were proposed in this pull request?

Just fixed the log place introduced by #11401

## How was this patch tested?

unit tests.

Author: Shixiong Zhu <[email protected]>

Closes #11432 from zsxwing/SPARK-13522-follow-up.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request May 12, 2016
## What changes were proposed in this pull request?

Just fixed the log place introduced by apache#11401

## How was this patch tested?

unit tests.

Author: Shixiong Zhu <[email protected]>

Closes apache#11432 from zsxwing/SPARK-13522-follow-up.

(cherry picked from commit ced71d3)
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.

3 participants