[KAFKA-9820] validateMessagesAndAssignOffsetsCompressed allocates batch iterator which is not used

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.6.0
• Component/s: core
• Assignee: Unassigned
• Reporter: Lucas Bradstreet
• Created: 04/Apr/20 00:05
• Updated: 04/Apr/20 17:10
• Resolved: 04/Apr/20 17:10

Description

KAFKA-8106 added a new skip key/value iterator that reduces allocations https://github.com/apache/kafka/commit/3e9d1c1411c5268de382f9dfcc95bdf66d0063a0.

Unfortunately in LogValidator it creates that iterator but it never uses it, and this is quite expensive in terms of allocations.

Issue Links

• links to: GitHub Pull Request #8422