Skip to content
This repository was archived by the owner on Jun 19, 2026. It is now read-only.

Add unit tests for the kafka api protocol, at the byte level#18

Open
awr wants to merge 1 commit into
gigya:masterfrom
NudgeSoftware:protocol_testing
Open

Add unit tests for the kafka api protocol, at the byte level#18
awr wants to merge 1 commit into
gigya:masterfrom
NudgeSoftware:protocol_testing

Conversation

@awr

@awr awr commented Aug 30, 2016

Copy link
Copy Markdown

Add support for:
OffsetCommitRequest (version 1 & 2)
ProduceRequest (version 1 & 2)
ProduceResponse (version 1 & 2)
Message versioning (using MagicNumber)
OffsetCommitRequest (version 1 & 2)

Add support for:
OffsetCommitRequest (version 1 & 2)
ProduceRequest (version 1 & 2)
ProduceResponse (version 1 & 2)
Message versioning (using MagicNumber)
OffsetCommitRequest (version 1 & 2)

# Conflicts:
#	src/KafkaNetClient/Common/BigEndianBinaryReader.cs
#	src/KafkaNetClient/Protocol/BaseRequest.cs
#	src/KafkaNetClient/Protocol/ProduceRequest.cs
#	src/kafka-net/Protocol/Protocol.cs
@EranOfer

Copy link
Copy Markdown

Hi,
Great job, it looks great!
It will take me a few days to go through it because I need to install Kafka Version 0.9

I'll keep you posted,

Eran

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants