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

Jason Gustafson commented on KAFKA-7549:
----------------------------------------

Sorry for the late response. As I understand it, the main issue is that we 
close the connection. Which error code we return seems a matter of taste (I 
think my preference is probably option 1 since it should only be client 
developers seeing this error and not end users, but I don't have a strong 
preference). I think probably it is a mistake to do the message format 
validation inside the ProduceRequest constructor. The current error handling 
logic depends on having a request object in order to return the proper 
response. Perhaps we should consider moving that validation into LogValidator?

> Old ProduceRequest with zstd compression does not return error to client
> ------------------------------------------------------------------------
>
>                 Key: KAFKA-7549
>                 URL: https://issues.apache.org/jira/browse/KAFKA-7549
>             Project: Kafka
>          Issue Type: Bug
>          Components: compression
>            Reporter: Magnus Edenhill
>            Assignee: Lee Dongjin
>            Priority: Major
>             Fix For: 2.2.0, 2.1.1
>
>
> Kafka broker v2.1.0rc0.
>  
> KIP-110 states that:
> "Zstd will only be allowed for the bumped produce API. That is, for older 
> version clients(=below KAFKA_2_1_IV0), we return UNSUPPORTED_COMPRESSION_TYPE 
> regardless of the message format."
>  
> However, sending a ProduceRequest V3 with zstd compression (which is a client 
> side bug) closes the connection with the following exception rather than 
> returning UNSUPPORTED_COMPRESSION_TYPE in the ProduceResponse:
>  
> {noformat}
> [2018-10-25 11:40:31,813] ERROR Exception while processing request from 
> 127.0.0.1:60723-127.0.0.1:60656-94 (kafka.network.Processor)
> org.apache.kafka.common.errors.InvalidRequestException: Error getting request 
> for apiKey: PRODUCE, apiVersion: 3, connectionId: 
> 127.0.0.1:60723-127.0.0.1:60656-94, listenerName: ListenerName(PLAINTEXT), 
> principal: User:ANONYMOUS
> Caused by: org.apache.kafka.common.record.InvalidRecordException: Produce 
> requests with version 3 are note allowed to use ZStandard compression
> {noformat}



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

Reply via email to