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

MINOR: Update dependencies for 0.11 #2872

Closed
wants to merge 1 commit into from

Conversation

ijuma
Copy link
Contributor

@ijuma ijuma commented Apr 19, 2017

Worth special mention:

  1. Update Scala to 2.11.11 and 2.12.2
  2. Update Gradle to 3.5
  3. Update ZooKeeper to 3.4.10
  4. Update reflections to 0.9.11, which:
    • Switches to jsr305 annotations with a provided scope
    • Updates Guava from 18 to 20
    • Updates javaassist from 3.18 to 3.21

There’s a separate PR for updating RocksDb, so
I didn’t include that here.

Worth special mention:

1. Update Scala to 2.11.11 and 2.12.2
2. Update Gradle to 3.5
3. Update ZooKeeper to 3.4.10
4. Update reflections to 0.9.11, which:
    * Switches to jsr305 annotations with a provided scope
    * Updates Guava from 18 to 20
    * Updates javaassist from 3.18 to 3.21

There’s a separate PR for updating RocksDb, so
I didn’t include that here.
@enothereska
Copy link
Contributor

Cool, LGTM

@asfbot
Copy link

asfbot commented Apr 19, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/3018/
Test PASSed (JDK 8 and Scala 2.11).

@asfbot
Copy link

asfbot commented Apr 19, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/3013/
Test PASSed (JDK 7 and Scala 2.10).

@ijuma
Copy link
Contributor Author

ijuma commented Apr 19, 2017

Review by @hachikuji

@asfbot
Copy link

asfbot commented Apr 19, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/3014/
Test PASSed (JDK 8 and Scala 2.12).

@hachikuji
Copy link
Contributor

LGTM. Might be worth running one of the system tests just to make sure nothing breaks in the provisioning process.

@ijuma
Copy link
Contributor Author

ijuma commented Apr 20, 2017

@asfgit asfgit closed this in c2d17c8 Apr 20, 2017
@ijuma ijuma deleted the update-deps-for-0.11 branch September 5, 2017 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants