gharris1727 commented on PR #13367: URL: https://github.com/apache/kafka/pull/13367#issuecomment-1468475024
> One high-level thought: it seems like we've elected to drop integration testing coverage for offset.lag.max = 0 and replace it with offset.lag.max = 10. Do you think there's any benefit in retaining at least one case where the max lag is 0? Before KAFKA-12468, we did not have offset.lag.max=0 coverage, so this is returning to the coverage we had before that patch. Since offset.lag.max=0 does have some effect, i'll leave it active in the transactional test. I didn't think it was valuable enough to justify a new test suite's runtime, but using it in an existing test shouldn't be too harmful. -- 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
