Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Dec 31, 2014

No description provided.

@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24970 has started for PR 3862 at commit 72121af.

  • This patch merges cleanly.

@rxin
Copy link
Contributor Author

rxin commented Dec 31, 2014

This subsumes #3649

@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24970 has finished for PR 3862 at commit 72121af.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • implicit class StrongToAttributeConversionHelper(val sc: StringContext)

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24970/
Test PASSed.

@culler
Copy link

culler commented Dec 31, 2014

Hi @rxin I think you meant to name the class StringToAttributeConversionHelper.

@rxin
Copy link
Contributor Author

rxin commented Dec 31, 2014

Thanks, @culler. Fixed.

@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24975 has started for PR 3862 at commit 9b10f57.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 31, 2014

Test build #24975 has finished for PR 3862 at commit 9b10f57.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • implicit class StringToAttributeConversionHelper(val sc: StringContext)

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/24975/
Test PASSed.

@rxin
Copy link
Contributor Author

rxin commented Jan 5, 2015

cc @marmbrus

@marmbrus
Copy link
Contributor

marmbrus commented Jan 5, 2015

LGTM

@marmbrus
Copy link
Contributor

marmbrus commented Jan 5, 2015

Thanks! Merged to master.

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