-
Notifications
You must be signed in to change notification settings - Fork 346
Bump spark version to 2.4.0 #670
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
Conversation
8f9b700
to
ffef3b5
Compare
tests are passed, but mimaReport is falling:
looks like is not related to this PR. |
algebird-spark/src/main/scala/com/twitter/algebird/spark/AlgebirdRDD.scala
Outdated
Show resolved
Hide resolved
ffef3b5
to
876ec37
Compare
@johnynek still falling with same error:
not sure, why it is looking for snapshot artifact.
we defently didn't release them for scala 2.12
did we release 0.13.4 version of |
876ec37
to
37aa645
Compare
we did not release algebird-spark for 2.12 because there was no spark for 2.12 until I think today. |
yes, that's error we can ignore, what's do you think about others? |
I think adding spark here: https://github.com/twitter/algebird/pull/670/files#diff-fdc3abdfd754eeb24090dbd90aeec2ceR159 then remove it after we publish the next version is probably the best fix as long as we are passing mima on 2.11. |
actually, maybe I didn't deploy at all because I couldn't figure out the sbt tricks since 2.12 wasn't supported, I may have just commented it out due to frustration. |
67bc2ba
to
c440074
Compare
c440074
to
17a2a8d
Compare
we actually hit sbt/sbt#4181 :) added workaround. |
Hey,
to support
algerbird-spark
#669 module on scala 2.12.x, we upgrade spark dependency to 2.4.0.also, we need to disable
-opt:l:inline
and-opt-inline-from:com.twitter.algebird.**
to workround issue scala/bug#11247.also, upgrade sbt script as the side effect.