Skip to content

Conversation

@kayousterhout
Copy link
Contributor

This is #4630 but modified for the 1.2 branch, because I'm guessing it makes sense to fix this issue in that branch (again, unless I missed something obvious here...)

@SparkQA
Copy link

SparkQA commented Feb 16, 2015

Test build #27574 has finished for PR 4631 at commit ffe8ff2.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@marmbrus
Copy link
Contributor

test this please

@SparkQA
Copy link

SparkQA commented Feb 17, 2015

Test build #27597 has finished for PR 4631 at commit ffe8ff2.

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

@marmbrus
Copy link
Contributor

/cc @liancheng

@liancheng
Copy link
Contributor

LGTM, thanks! Merging into branch-1.2.

asfgit pushed a commit that referenced this pull request Feb 19, 2015
This is #4630 but modified for the 1.2 branch, because I'm guessing it makes sense to fix this issue in that branch (again, unless I missed something obvious here...)

Author: Kay Ousterhout <[email protected]>

Closes #4631 from kayousterhout/SPARK-5846_1.2.1 and squashes the following commits:

ffe8ff2 [Kay Ousterhout] [SPARK-5846] Correctly set job description and pool for SQL jobs
@marmbrus
Copy link
Contributor

Mind closing this now?

markhamstra pushed a commit to markhamstra/spark that referenced this pull request Feb 24, 2015
This is apache#4630 but modified for the 1.2 branch, because I'm guessing it makes sense to fix this issue in that branch (again, unless I missed something obvious here...)

Author: Kay Ousterhout <[email protected]>

Closes apache#4631 from kayousterhout/SPARK-5846_1.2.1 and squashes the following commits:

ffe8ff2 [Kay Ousterhout] [SPARK-5846] Correctly set job description and pool for SQL jobs
@kayousterhout kayousterhout deleted the SPARK-5846_1.2.1 branch April 12, 2017 00:44
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