[KAFKA-5052] We shouldn't pass the underlying exception to RetriableCommitFailedException when an async offset commit fails.

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.11.0.0
• Assignee: Apurva Mehta
• Reporter: Apurva Mehta
• Created: 10/Apr/17 20:56
• Updated: 13/Jun/17 23:17
• Resolved: 11/Apr/17 00:21

Description

• This is a follow up to : https://issues.apache.org/jira/browse/KAFKA-3527

We currently wrap retriable exceptions encountered during offset commits in a `RetriableOffsetCommitException`. The problem is that today we also pass the underlying internal exception on to the user. There isn't a really good reason to do this, since the user will not handle each individual exception differently: they will just retry anyway.

We should not pass on the underlying internal exception. It makes the API simpler, and also allows us to change things underneath with more flexibility.

Issue Links

• links to: GitHub Pull Request #2838