Skip to content

Conversation

@liancheng
Copy link
Contributor

What changes were proposed in this pull request?

This PR backports PR #16866 to branch-1.6

How was this patch tested?

Existing tests.

@SparkQA
Copy link

SparkQA commented Feb 14, 2017

Test build #72835 has finished for PR 16917 at commit f7d7c54.

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

@liancheng
Copy link
Contributor Author

cc @JoshRosen

@JoshRosen
Copy link
Contributor

LGTM, so I'm going to merge this to branch-1.6. Thanks for putting together the backport!

asfgit pushed a commit that referenced this pull request Feb 14, 2017
## What changes were proposed in this pull request?

This PR backports PR #16866 to branch-1.6

## How was this patch tested?

Existing tests.

Author: Cheng Lian <[email protected]>

Closes #16917 from liancheng/spark-19529-1.6-backport.
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Feb 15, 2017
## What changes were proposed in this pull request?

This PR backports PR apache#16866 to branch-1.6

## How was this patch tested?

Existing tests.

Author: Cheng Lian <[email protected]>

Closes apache#16917 from liancheng/spark-19529-1.6-backport.

(cherry picked from commit a50ef3d)
@yhuai
Copy link
Contributor

yhuai commented Feb 28, 2017

Let's use a meaningful title in future :)

@liancheng liancheng closed this Mar 13, 2017
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