amirdeljouyi opened a new pull request, #22757: URL: https://github.com/apache/kafka/pull/22757
Hi, I added focused regression coverage for offset reset and validation retry behavior. I also tightened the existing truncation tests with assertions for the string description. Impact on coverage: * Covers the relevant SubscriptionState logic, including the `maybeSeekUnvalidated` [reset guards](https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/internals/SubscriptionState.java#L449-L461) and the [validation retry paths](https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/clients/consumer/internals/SubscriptionState.java#L893-L913) * Branch coverage for SubscriptionState increases by around 15%. Why: These checks make sure partitions only become eligible for validation/reset at the right time and that truncation details stay visible when they are reported. I hope you find these tests useful. 😄 Please let me know if you have any questions or concerns. Thanks, -- 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]
