[WIP] Scala 2.13 compatible build#860
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. |
| import ml.combust.mleap.core.annotation.SparkCode | ||
|
|
||
| import scala.collection.JavaConverters._ | ||
| import scala.collection.parallel.CollectionConverters._ |
There was a problem hiding this comment.
@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. |
| language: scala | ||
| scala: | ||
| - 2.12.18 | ||
| - 2.13.11 |
There was a problem hiding this comment.
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.