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

KAFKA-5076: remove usage of java.xml.bind.* classes #2862

Closed
wants to merge 1 commit into from

Conversation

xvrl
Copy link
Member

@xvrl xvrl commented Apr 17, 2017

  • replaces base64 from DatatypeConverter with Base64 introduced in JDK 8
  • this change enables running Kafka with Java 9

Given that we plan to stop supporting Java 7 in 0.11 this should be fine, but merging this would depend on when we are comfortable introducing backwards-incompatible changes.

* replaces base64 from DatatypeConverter with Base64 introduced in jdk 8
@asfbot
Copy link

asfbot commented Apr 17, 2017

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

@asfbot
Copy link

asfbot commented Apr 17, 2017

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

@asfbot
Copy link

asfbot commented Apr 17, 2017

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

@ijuma
Copy link
Member

ijuma commented Jul 20, 2017

We decided not to move to Java 8 for 0.11 in the end. So we may need another solution.

@asfgit
Copy link

asfgit commented Jul 20, 2017

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

@asfgit
Copy link

asfgit commented Jul 20, 2017

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

@ijuma
Copy link
Member

ijuma commented Aug 9, 2017

@xvrl #3647 includes a solution that works in Java 7 and Java 9.

@asfgit
Copy link

asfgit commented Aug 9, 2017

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

@asfgit
Copy link

asfgit commented Aug 9, 2017

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

@xvrl
Copy link
Member Author

xvrl commented Aug 16, 2017

sounds good

@xvrl xvrl closed this Aug 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants