Skip to content

Conversation

@sarutak
Copy link
Member

@sarutak sarutak commented Sep 3, 2014

No description provided.

@srowen
Copy link
Member

srowen commented Sep 3, 2014

A quick search shows 10 other occurrences of "SparkSQL" in comments and docs. If you're bothering with this as a PR and JIRA, maybe hit them all?

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have started for PR 2251 at commit 6df66de.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have finished for PR 2251 at commit 6df66de.

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

@sarutak
Copy link
Member Author

sarutak commented Sep 3, 2014

@srowen Thanks, I replaced rest of "SparkSQL" with "Spark SQL" without application name.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have started for PR 2251 at commit bb5938f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have finished for PR 2251 at commit bb5938f.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • protected trait YarnAllocateResponse

@liancheng
Copy link
Contributor

LGTM, thanks :)

@liancheng
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Sep 3, 2014

QA tests have started for PR 2251 at commit bb5938f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have finished for PR 2251 at commit bb5938f.

  • This patch fails unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class SparkListenerBlockManagerAdded(time: Long, blockManagerId: BlockManagerId, maxMem: Long)
    • case class SparkListenerBlockManagerRemoved(time: Long, blockManagerId: BlockManagerId)
    • case class SparkListenerApplicationStart(appName: String, appId: Option[String], time: Long,
    • protected trait YarnAllocateResponse

@sarutak
Copy link
Member Author

sarutak commented Sep 4, 2014

retest this please

@JoshRosen
Copy link
Contributor

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have started for PR 2251 at commit 0bfe234.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 4, 2014

QA tests have finished for PR 2251 at commit 0bfe234.

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

@marmbrus
Copy link
Contributor

marmbrus commented Sep 4, 2014

Thanks! Merged to master. :)

BTW, no need to create a JIRA for doc updates.

@asfgit asfgit closed this in dc1ba9e Sep 4, 2014
@sarutak sarutak deleted the SPARK-3378 branch April 11, 2015 05:21
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.

6 participants