Skip to content

Conversation

@HyukjinKwon
Copy link
Member

The main purpose of this PR is to backport #9754

I added several commits but they are identical with the PR.

I will cc @liancheng just to find this easily.

@HyukjinKwon
Copy link
Member Author

I mistakenly added SPARK-11692 (#9658) here as I though this is supposed to go to branch 1.6.0 but this is classified for version 1.7.

I will take this out.

@HyukjinKwon HyukjinKwon changed the title [SPARK-11692] [SPARK-11694] [SQL] Backports #9658 and #9754 [SPARK-11694] [SQL] Backports #9754 Nov 17, 2015
@liancheng
Copy link
Contributor

LGTM pending Jenkins.

@SparkQA
Copy link

SparkQA commented Nov 17, 2015

Test build #46075 has finished for PR 9763 at commit 1f252ec.

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

@SparkQA
Copy link

SparkQA commented Nov 17, 2015

Test build #46082 has finished for PR 9763 at commit bedb89a.

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

@liancheng
Copy link
Contributor

Thanks! Merging to branch-1.6.

BTW, it's not a big deal, but you can cc other users in comments instead of in PR descriptions. Because the PR description will be part of the final commit message when a PR is merged.

asfgit pushed a commit that referenced this pull request Nov 17, 2015
The main purpose of this PR is to backport #9754

I added several commits but they are identical with the PR.

I will cc liancheng just to find this easily.

Author: hyukjinkwon <gurwls223@gmail.com>
Author: HyukjinKwon <gurwls223@gmail.com>

Closes #9763 from HyukjinKwon/SPARK-11694-followup-backporting.
@JoshRosen
Copy link
Contributor

Hi @HyukjinKwon, would you mind closing this PR on GitHub now that it's been merged? GitHub won't auto-close backport branch PRs like this one.

@HyukjinKwon
Copy link
Member Author

Oh, sure. Thanks!
Closing this.

@HyukjinKwon HyukjinKwon deleted the SPARK-11694-followup-backporting branch September 23, 2016 18:28
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