[KAFKA-14916] Fix code that assumes transactional ID implies all records are transactional

• Type: Sub-task
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 3.6.0
• Fix Version/s: 3.5.0
• Assignee: Justine Olshan
• Reporter: Justine Olshan
• Created: 17/Apr/23 23:11
• Updated: 20/Sep/23 04:20
• Resolved: 04/May/23 17:28

Description

KAFKA-14561 wrote code that assumed that if a transactional ID was included, all record batches were transactional and had the same producer ID.

This work with improve validation and fix the code that assumes all batches are transactional.
Further, KAFKA-14561 will not assume all records are transactional.

Originally this ticket had an action item to ensure all the producer IDs are the same in the batches since we send a single txn ID, but that can be done in a followup KAFKA-14958, as we still need to assess if we can enforce this without breaking workloads.

Issue Links

• links to: GitHub Pull Request #13607