Skip to content

Conversation

@wzhfy
Copy link
Contributor

@wzhfy wzhfy commented Jun 25, 2017

What changes were proposed in this pull request?

Builds failed due to the recent merge. This is because PR#18309 needed update after this patch was merged.

How was this patch tested?

N/A

@wzhfy
Copy link
Contributor Author

wzhfy commented Jun 25, 2017

cc @srowen @cloud-fan

@srowen
Copy link
Member

srowen commented Jun 25, 2017

@wzhfy so this needs to go into master/2.2?

@SparkQA
Copy link

SparkQA commented Jun 25, 2017

Test build #78573 has finished for PR 18415 at commit 6c3b7d4.

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

@wzhfy
Copy link
Contributor Author

wzhfy commented Jun 25, 2017

retest this please

@wzhfy
Copy link
Contributor Author

wzhfy commented Jun 25, 2017

@srowen Only master, tasks of removing sql conf were only merged in master.

@SparkQA
Copy link

SparkQA commented Jun 25, 2017

Test build #78577 has finished for PR 18415 at commit 6c3b7d4.

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

@viirya
Copy link
Member

viirya commented Jun 25, 2017

LGTM

@srowen
Copy link
Member

srowen commented Jun 25, 2017

merged to master

@asfgit asfgit closed this in 884347e Jun 25, 2017
robert3005 pushed a commit to palantir/spark that referenced this pull request Jun 29, 2017
## What changes were proposed in this pull request?

Builds failed due to the recent [merge](apache@b449a1d). This is because [PR#18309](apache#18309) needed update after [this patch](apache@b803b66) was merged.

## How was this patch tested?
N/A

Author: Zhenhua Wang <wzh_zju@163.com>

Closes apache#18415 from wzhfy/hotfixStats.
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