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

Jun Rao commented on KAFKA-349:
-------------------------------

Thanks for patch v5. One more comment:

51. FetchResponse,ProducerResponse: We will still need to send the versionId to 
the client so that the client knows if it gets the right version of the 
response. We just don't need CurrentVersion in FetchResponse/ProducerResponse 
since we should just refer to the version constant in 
FetchRequest/ProducerRequest.

                
> Create individual "Response" types for each kind of request and wrap them 
> with "BoundedByteBufferSend", remove "xxResponseSend" types for all requests 
> except "FetchRequest"
> ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: KAFKA-349
>                 URL: https://issues.apache.org/jira/browse/KAFKA-349
>             Project: Kafka
>          Issue Type: Improvement
>            Reporter: Yang Ye
>            Assignee: Yang Ye
>         Attachments: kafka_responseSend.patch, kafka_responseSend.patch.2, 
> kafka_responseSend.patch.3, kafka_responseSend.patch.4, 
> kafka_responseSend.patch.5
>
>


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to