hgeraldino opened a new pull request, #13386: URL: https://github.com/apache/kafka/pull/13386
This shouldn't be a controversial change: the `if` condition here is checking the wrong variable (`this.committableOffsets.isEmpty()` always evaluates to true because of L#216). I assume this bug went undetected because it's only used to log information around committable offsets. Should I file a JIRA? ### Committer Checklist (excluded from commit message) - [ ] Verify design and implementation - [ ] Verify test coverage and CI build status - [ ] Verify documentation (including upgrade notes) -- 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