Jason, Colin, and Mathias, do you have any comments on the KIP as it stands now?
Thank you. Sarwar On Fri, 23 Apr 2021 at 10:12, Sarwar Bhuiyan <sar...@confluent.io> wrote: > Hi all, > > I'm reviving this discussion after perusing the vote thread created by > Stan. I believe the KIP is already revised to include Jason and Colin's > suggestion of using a RecordDeserializationException. It signals to the > user that something needs to be done (whether skipping the record as shown > in the example and/or do something else). This is backward compatible with > what we have today and is clean. > > Mathias suggested that we split the exception types into a > RecordDeserializationException and a CorruptRecordException with the > intention of auto-skipping corrupt records. I would prefer to use another > KIP for that type of functionality. > > KIP: > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=87297793 > > JIRAs: > https://issues.apache.org/jira/browse/KAFKA-5682 > > Last Vote thread comment from Stan: > https://www.mail-archive.com/dev@kafka.apache.org/msg92392.html > > On another note, I need edit access to the KIP page. Could somebody give > me access should I need to make changes? My username is sarwarb. > > Thanks. > > Sarwar > > > > -- [image: Confluent] <https://www.confluent.io/> Sarwar Bhuiyan Staff Customer Success Technical Architect +447949684437 Follow us: [image: Blog] <https://www.confluent.io/blog?utm_source=footer&utm_medium=email&utm_campaign=ch.email-signature_type.community_content.blog>[image: Twitter] <https://twitter.com/ConfluentInc>[image: LinkedIn] <https://www.linkedin.com/company/confluent/>[image: Slack] <https://slackpass.io/confluentcommunity>[image: YouTube] <https://youtube.com/confluent> [image: Kafka Summit] <https://www.kafka-summit.org/>
