-
Notifications
You must be signed in to change notification settings - Fork 312
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
[WIP] Scala 2.13 compatible build #860
base: master
Are you sure you want to change the base?
Conversation
Think you need to alter the travis yaml to make scala 2.13 available https://github.com/combust/mleap/blob/master/.travis.yml#L25 |
9aae3cd
to
3061cc5
Compare
Hi Thanks |
Hi Thanks |
@db-scnakandala any rough timeline on your end? I'm happy to merge and release artifacts soon after the PR is complete |
Didn't get a chance to go through the build failures. Will check again this week. |
@db-scnakandala Hey, any updates on this? Is there something I can do to help you? |
@jsleight Hey, you mentioned that updating the travis.yml to 2.13.11 should make it compatible to scala 2.13? Because I did the same the local compile was successful. I see this PR is taking quite some time and I need to make changes so we can make it compatible. Please update, whenever you do get a chance as it would be really helpful. |
68d2672
to
00f2fef
Compare
00f2fef
to
993896e
Compare
@prashil1996 I am still trying to diagnose and fix several CI failures. Please let me know if you have any insights on the test failures. |
@db-scnakandala Let me look into the build failures. I see that the Mleap Executor & Mleap benchmark have passed. Let me look into the logs and come back to you. |
@@ -23,6 +23,7 @@ import java.util.UUID | |||
import ml.combust.mleap.core.annotation.SparkCode | |||
|
|||
import scala.collection.JavaConverters._ | |||
import scala.collection.parallel.CollectionConverters._ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@db-scnakandala There is an error while importing has this has been moved in Scala 2.13 to separate module [scala/scala-parallel-collection]
Need to update the library dependency such as for scala 2.13 onwards.
libraryDependencies += "org.scala-lang.modules" %% "scala-parallel-collections" % "1.0.0"
This is where one build is failing.
[error] /home/travis/build/combust/mleap/mleap-spark-extension/src/main/scala/org/apache/spark/ml/mleap/classification/OneVsRest.scala:26:34: object CollectionConverters is not a member of package scala.collection.parallel [error] import scala.collection.parallel.CollectionConverters._
@db-scnakandala Also, I was also facing this issue locally where the following were not found. Is there a way to resolve this dependency? @jsleight |
And the other major error is due to NoSuchMethodError error. |
That link is giving the minimum versions. So any scala 2.13.x version should work with jdk11. It matters more for older scala versions -- e.g., you need scala 2.12.4 or newer for a scala 2.12 (i.e., scala 2.12.3 wouldn't work with jdk11). Our spark version is compiled with scala 2.13.8 -- I'd expect any scala 2.13 version to work though. |
We have xgboost 1.7.3, so I'm not sure why you would be seeing errors for xgboost 2.0, let alone for a snapshot version. Would likely be something wrong with your branch. |
- 2.12.18 | ||
- 2.13.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we'll need to have both a scala 2.12 and a scala 2.13 distribution installed in travis (so the cross builds can run). Refer to travis docs for how to do this.
@db-scnakandala Any updates? |
No description provided.