sebastienviale commented on code in PR #16093:
URL: https://github.com/apache/kafka/pull/16093#discussion_r1618714738


##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/RecordQueue.java:
##########
@@ -232,12 +236,14 @@ private void updateHead() {
             }
             headRecord = new StampedRecord(deserialized, timestamp);
             headRecordSizeInBytes = consumerRecordSizeInBytes(raw);
+            rawHeadRecord = raw;

Review Comment:
   We considered putting the rawRecord into the StampedRecord, but this would 
have required changes to some signature methods.
   This is why we chose to do it this way. 



##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/StreamTask.java:
##########
@@ -764,6 +766,7 @@ public boolean process(final long wallClockTime) {
 
             // get the next record to process
             record = partitionGroup.nextRecord(recordInfo, wallClockTime);
+            rawRecord = partitionGroup.rawHeadRecord();

Review Comment:
   same as above



-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to