Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-7011] Build(compilation) fails with scala 2.11 option, because a protected[sql] type is accessed in ml package. #5593

Closed
wants to merge 1 commit into from

Conversation

ScrapCodes
Copy link
Member

This is where it is used and fails compilations at.

@SparkQA
Copy link

SparkQA commented Apr 20, 2015

Test build #30586 has started for PR 5593 at commit e6d57a3.

@srowen
Copy link
Member

srowen commented Apr 20, 2015

Seems OK to me.

@ScrapCodes
Copy link
Member Author

@mengxr and @marmbrus PTAL.

@SparkQA
Copy link

SparkQA commented Apr 20, 2015

Test build #30586 has finished for PR 5593 at commit e6d57a3.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • protected[spark] abstract class NativeType extends DataType
  • This patch does not change any dependencies.

@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/30586/
Test PASSed.

@marmbrus
Copy link
Contributor

This is fine to fix the build break, but ideally MLlib would not rely on this internal detail (that I'd like to remove). @mengxr, what are you guys using this for?

@asfgit asfgit closed this in 04bf34e Apr 21, 2015
@marmbrus
Copy link
Contributor

Thanks, merged to master.

@mengxr, we should figure out how you are using NativeType as I would like to get rid of it.

@ScrapCodes ScrapCodes deleted the SPARK-7011/build-fix branch June 3, 2015 05:52
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
… a protected[sql] type is accessed in ml package.

[This](https://github.com/apache/spark/blob/master/mllib/src/main/scala/org/apache/spark/ml/feature/VectorAssembler.scala#L58) is where it is used and fails compilations at.

Author: Prashant Sharma <prashant.s@imaginea.com>

Closes apache#5593 from ScrapCodes/SPARK-7011/build-fix and squashes the following commits:

e6d57a3 [Prashant Sharma] [SPARK-7011] Build fails with scala 2.11 option, because a protected[sql] type is accessed in ml package.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants