Skip to content

[LI-CHERRY-PICK] [f8129f6f] KAFKA-9203: Revert "MINOR: Remove workaro…#84

Merged
xiowu0 merged 1 commit into
linkedin:2.3-li-1from
xiowu0:kafka-9203
Jun 3, 2020
Merged

[LI-CHERRY-PICK] [f8129f6f] KAFKA-9203: Revert "MINOR: Remove workaro…#84
xiowu0 merged 1 commit into
linkedin:2.3-li-1from
xiowu0:kafka-9203

Conversation

@xiowu0

@xiowu0 xiowu0 commented Jun 3, 2020

Copy link
Copy Markdown

…unds for lz4-java bug affecting byte buffers (apache#6679)" (apache#7769)

TICKET = KAFKA-9203
LI_DESCRIPTION =

EXIT_CRITERIA = HASH [f8129f6]
ORIGINAL_DESCRIPTION =

This reverts commit 90043d5 as it caused a regression in some cases:

Caused by: java.io.IOException: Stream frame descriptor corrupted
at org.apache.kafka.common.record.KafkaLZ4BlockInputStream.readHeader(KafkaLZ4BlockInputStream.java:132)
at org.apache.kafka.common.record.KafkaLZ4BlockInputStream.(KafkaLZ4BlockInputStream.java:78)
at org.apache.kafka.common.record.CompressionType$4.wrapForInput(CompressionType.java:110)

I will investigate why after, but I want to get the safe fix into 2.4.0.
The reporter of KAFKA-9203 has verified that reverting this change
makes the problem go away.

Reviewers: Manikumar Reddy manikumar.reddy@gmail.com
(cherry picked from commit f8129f6)

…nds for lz4-java bug affecting byte buffers (apache#6679)" (apache#7769)

TICKET = KAFKA-9203
LI_DESCRIPTION =

EXIT_CRITERIA = HASH [f8129f6]
ORIGINAL_DESCRIPTION =

This reverts commit 90043d5 as it caused a regression in some cases:

> Caused by: java.io.IOException: Stream frame descriptor corrupted
>         at org.apache.kafka.common.record.KafkaLZ4BlockInputStream.readHeader(KafkaLZ4BlockInputStream.java:132)
>         at org.apache.kafka.common.record.KafkaLZ4BlockInputStream.<init>(KafkaLZ4BlockInputStream.java:78)
>         at org.apache.kafka.common.record.CompressionType$4.wrapForInput(CompressionType.java:110)

I will investigate why after, but I want to get the safe fix into 2.4.0.
The reporter of KAFKA-9203 has verified that reverting this change
makes the problem go away.

Reviewers: Manikumar Reddy <manikumar.reddy@gmail.com>
(cherry picked from commit f8129f6)
@xiowu0 xiowu0 merged commit 9c16d8c into linkedin:2.3-li-1 Jun 3, 2020
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.

3 participants