mleap icon indicating copy to clipboard operation
mleap copied to clipboard

[WIP] Scala 2.13 compatible build

Open db-scnakandala opened this issue 1 year ago • 14 comments

db-scnakandala avatar Jul 10 '23 20:07 db-scnakandala

Think you need to alter the travis yaml to make scala 2.13 available https://github.com/combust/mleap/blob/master/.travis.yml#L25

jsleight avatar Jul 10 '23 20:07 jsleight

Hi Is there any ETA on when this PR gets merged and 2.13 mleap libraries will be available?

Thanks

scheruku-in avatar Jul 18 '23 16:07 scheruku-in

Hi Is there any ETA on when this PR gets merged and 2.13 mleap libraries will be available?

Thanks

scheruku-in avatar Aug 02 '23 07:08 scheruku-in

@db-scnakandala any rough timeline on your end? I'm happy to merge and release artifacts soon after the PR is complete

jsleight avatar Aug 02 '23 15:08 jsleight

@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 avatar Aug 02 '23 17:08 db-scnakandala

@db-scnakandala Hey, any updates on this? Is there something I can do to help you?

prashil1996 avatar Oct 15 '23 20:10 prashil1996

@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. Screenshot 2023-10-16 at 2 01 26 AM

Please update, whenever you do get a chance as it would be really helpful.

prashil1996 avatar Oct 15 '23 20:10 prashil1996

@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 avatar Oct 16 '23 06:10 db-scnakandala

@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.

prashil1996 avatar Oct 16 '23 06:10 prashil1996

@db-scnakandala Also, I was also facing this issue locally where the following were not found. https://s3-us-west-2.amazonaws.com/xgboost-maven-repo/snapshot/ml/dmlc/xgboost4j_2.12/2.0.0-SNAPSHOT/xgboost4j_${scala.binary.version}-2.0.0-20230817.090749-594.jar: not found: https://s3-us-west-2.amazonaws.com/xgboost-maven-repo/snapshot/ml/dmlc/xgboost4j_2.12/2.0.0-SNAPSHOT/xgboost4j_${scala.binary.version}-2.0.0-20230817.090749-594.jar

Is there a way to resolve this dependency? @jsleight

prashil1996 avatar Oct 16 '23 06:10 prashil1996

And the other major error is due to NoSuchMethodError error. Exception in thread "main" java.lang.NoSuchMethodError: scala.util.matching.Regex.<init>(Ljava/lang/String;Lscala/collection/Seq;)V Can we make sure that the spark version we are using for scala 2.13.11, if its compatible or not? Becase the jdk version is at openjdk11 and the scala version being passed is 2.13.11, but as per the documentation openjdk11 supports only 2.13.0, so maybe there could be a version mismatch there. https://docs.scala-lang.org/overviews/jdk-compatibility/overview.html @db-scnakandala

prashil1996 avatar Oct 16 '23 07:10 prashil1996

Can we make sure that the spark version we are using for scala 2.13.11, if its compatible or not? Becase the jdk version is at openjdk11 and the scala version being passed is 2.13.11, but as per the documentation openjdk11 supports only 2.13.0, so maybe there could be a version mismatch there. https://docs.scala-lang.org/overviews/jdk-compatibility/overview.html

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.

jsleight avatar Oct 16 '23 20:10 jsleight

@db-scnakandala Also, I was also facing this issue locally where the following were not found. https://s3-us-west-2.amazonaws.com/xgboost-maven-repo/snapshot/ml/dmlc/xgboost4j_2.12/2.0.0-SNAPSHOT/xgboost4j_${scala.binary.version}-2.0.0-20230817.090749-594.jar: not found: https://s3-us-west-2.amazonaws.com/xgboost-maven-repo/snapshot/ml/dmlc/xgboost4j_2.12/2.0.0-SNAPSHOT/xgboost4j_${scala.binary.version}-2.0.0-20230817.090749-594.jar

Is there a way to resolve this dependency? @jsleight

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.

jsleight avatar Oct 16 '23 20:10 jsleight

@db-scnakandala Any updates?

prashil1996 avatar Oct 20 '23 04:10 prashil1996