Skip to content

KAFKA-4783: Add ByteArrayConverter (KIP-128)#2599

Closed
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-4783-byte-array-converter
Closed

KAFKA-4783: Add ByteArrayConverter (KIP-128)#2599
ewencp wants to merge 1 commit intoapache:trunkfrom
ewencp:kafka-4783-byte-array-converter

Conversation

@ewencp
Copy link
Copy Markdown
Contributor

@ewencp ewencp commented Feb 25, 2017

No description provided.

@asfbot
Copy link
Copy Markdown

asfbot commented Feb 25, 2017

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

@asfbot
Copy link
Copy Markdown

asfbot commented Feb 25, 2017

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

@asfbot
Copy link
Copy Markdown

asfbot commented Feb 25, 2017

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

@pradeepcheers
Copy link
Copy Markdown

Could you please let me know the difference between org.apache.kafka.connect.converters.ByteArrayConverter‌ and io.confluent.connect.replicator.util.ByteArrayConverte‌​r? Does both these classes serve the same purpose?

@ewencp
Copy link
Copy Markdown
Contributor Author

ewencp commented Jul 6, 2017

@pradeepcheers There is no functional difference. We added a version to Kafka itself so it would be available across all connectors -- we saw a number of connectors including their own, so it made sense to include it out of the box since it's really simple anyway and should have no real maintenance cost.

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