mjsax commented on code in PR #17919:
URL: https://github.com/apache/kafka/pull/17919#discussion_r1857256843
##########
streams/src/main/java/org/apache/kafka/streams/processor/internals/PartitionGroup.java:
##########
@@ -250,10 +250,11 @@ StampedRecord nextRecord(final RecordInfo info, final
long wallClockTime) {
if (queue != null) {
// get the first record from this queue.
+ final int oldSize = queue.size();
record = queue.poll(wallClockTime);
if (record != null) {
- --totalBuffered;
+ totalBuffered -= oldSize - queue.size();
Review Comment:
> I think I get it now, under the covers poll may skip records due to
deserialization errors so we need to account for this in the totalBuffered
variable.
Exactly. -- Either deserialization errors, or invalid (negative) timestamp
extraction.
--
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
