Skip to content

Conversation

@carsonwang
Copy link
Contributor

This is to follow up #6834 , update the driver log URL as well for consistency.

@srowen
Copy link
Member

srowen commented Jun 18, 2015

Do you need to update YarnClusterSuite too? looks like it expects start=0

@carsonwang
Copy link
Contributor Author

Thanks for the reminder, @srowen ! The url in YarnClusterSuite is now updated.

@SparkQA
Copy link

SparkQA commented Jun 18, 2015

Test build #35118 has finished for PR 6878 at commit a0004f4.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class CreateStruct(children: Seq[Expression]) extends Expression

@SparkQA
Copy link

SparkQA commented Jun 18, 2015

Test build #35120 has finished for PR 6878 at commit 13be948.

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

@harishreedharan
Copy link
Contributor

As long as -4096 works (I didn't test this), this LGTM.

@andrewor14
Copy link
Contributor

LGTM, merging into master.

@asfgit asfgit closed this in 54557f3 Jun 19, 2015
@carsonwang carsonwang deleted the logUrl branch August 17, 2015 01:24
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.

5 participants