[jira] [Commented] (KAFKA-5682) Consumer should include partition in exceptions raised during record parsing/validation

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/KAFKA-5682?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16949645#comment-16949645 ] 

ASF GitHub Bot commented on KAFKA-5682:
---------------------------------------

stanislavkozlovski commented on pull request #7499: KAFKA-5682: Include partitions in exceptions raised during consumer record deserialization/validation
URL: https://github.com/apache/kafka/pull/7499
 
 
   This patch adds a new RecordDeserializationException which is raised when the user is expected to seek past an offset whose record is corrupted in some way
   
 
----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Consumer should include partition in exceptions raised during record parsing/validation
> ---------------------------------------------------------------------------------------
>
>                 Key: KAFKA-5682
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5682
>             Project: Kafka
>          Issue Type: Improvement
>          Components: consumer
>            Reporter: Jason Gustafson
>            Assignee: Stanislav Kozlovski
>            Priority: Minor
>              Labels: needs-kip
>
> When we encounter an exception when validating a fetched record or when deserializing it, we raise it to the user and keep the consumer's current position at the offset of the failed record. The expectation is that the user will either propagate the exception and shutdown or seek past the failed record. However, in the latter case, there is no way for the user to know which topic partition had the failed record. We should consider exposing an exception type to expose this information which users can catch. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)