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

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

vvcephei commented on pull request #6641: KAFKA-8254: Pass Changelog as Topic 
in Suppress Serdes (#6602)
URL: https://github.com/apache/kafka/pull/6641
 
 
   Cherry-picked from #6602 
   
   Reviewers:  Matthias J. Sax <matth...@confluent.io>, Guozhang Wang 
<wangg...@gmail.com>
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   
 
----------------------------------------------------------------
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:
us...@infra.apache.org


> Suppress incorrectly passes a null topic to the serdes
> ------------------------------------------------------
>
>                 Key: KAFKA-8254
>                 URL: https://issues.apache.org/jira/browse/KAFKA-8254
>             Project: Kafka
>          Issue Type: Bug
>          Components: streams
>    Affects Versions: 2.1.0, 2.2.0, 2.1.1
>            Reporter: John Roesler
>            Assignee: John Roesler
>            Priority: Major
>             Fix For: 2.3.0, 2.2.1
>
>
> For example, in KTableSuppressProcessor, we do:
> {noformat}
> final Bytes serializedKey = Bytes.wrap(keySerde.serializer().serialize(null, 
> key));
> {noformat}
> This violates the contract of Serializer (and Deserializer), and breaks 
> integration with known Serde implementations.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to