Skip to content

Conversation

crepererum
Copy link
Collaborator

@crepererum crepererum commented Feb 17, 2022

The only issue found by this is that we were not able to read
Snappy-compressed data produced by Java clients, because Java uses its
own weird framing format. Funnily Java clients can read the Snappy data
what we produce. This is also in line with the rdkafka implemention
(they can consume the "normal" and the special "Java" format but only
produce the "normal" one).

Closes #98.

@crepererum crepererum force-pushed the crepererum/java_interopt branch from db8ca1d to 67fa4ce Compare February 17, 2022 11:21
@crepererum crepererum marked this pull request as ready for review February 17, 2022 11:26
Copy link
Contributor

@tustvold tustvold left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very cool 👍

crepererum and others added 3 commits April 21, 2022 12:39
The only issue found by this is that we were not able to read
Snappy-compressed data produced by Java clients, because Java uses its
own weird framing format. Funnily Java clients can read the Snappy data
what we produce. This is also in line with the rdkafka implemention
(they can consume the "normal" and the special "Java" format but only
produce the "normal" one).
Co-authored-by: Raphael Taylor-Davies <[email protected]>
@crepererum crepererum force-pushed the crepererum/java_interopt branch from aae7555 to 47f2c12 Compare April 21, 2022 10:57
@crepererum crepererum added the automerge Instruct kodiak to merge the PR label Apr 21, 2022
@kodiakhq kodiakhq bot merged commit ad8de3c into main Apr 21, 2022
@kodiakhq kodiakhq bot deleted the crepererum/java_interopt branch April 21, 2022 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Instruct kodiak to merge the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test against java kafka client
2 participants