Doesn't work with new sarama
Serhii1011010 opened this issue · 1 comments
Serhii1011010 commented
Steps to reproduce:
1 clone this repository
2 go to _examples/sarama
dir
3 modify it to be up to date
4 go run main.go
everything is fine so far
5 change go.mod's sarama version to, say, v1.26.1
6 get out of memory error
The error is happening here https://github.com/travisjeffery/jocko/blob/master/protocol/decoder.go#L277 because this conversion https://github.com/travisjeffery/jocko/blob/master/protocol/decoder.go#L266 makes n
to be 4294967295
, which is probably because sarama changed endianness in their protocol.
Serhii1011010 commented
The issue appears in 1.17.0 version of sarama. 1.16.0 still works.