smjn commented on code in PR #22479:
URL: https://github.com/apache/kafka/pull/22479#discussion_r3366981087
##########
server/src/main/java/org/apache/kafka/server/share/dlq/ShareGroupDLQStateManager.java:
##########
@@ -231,6 +251,7 @@ public ProduceRequestHandler(
) {
this.param = param;
this.result = result;
+ this.originalRecordData = new ArrayList<>();
Review Comment:
Hi,
I realised there could be a bug here if the offsets are sparsely filled due
to compaction on the user topic - replacing this DS with a hashmap and adding
the suggested non-null check should be sufficient.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
