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

Guozhang Wang commented on KAFKA-1644:
--------------------------------------

Anton, I think Jun's not saying that there is a blocker for doing so, it's just 
that since FetchResponse does not use writeTo to write to the socket, but use 
FetchReponseSend instead for zero-copy, it may not logically be treated as 
inheritance of RequestOrResponse. After looking into your patch I see you are 
enforcing this by letting FetchResponseSend to just call FetchResponse.writeTo 
instead, which I think is OK if we do have a strong reason for letting 
FetchResponse inheriting from RequestOrResponse.

> Inherit FetchResponse from RequestOrResponse
> --------------------------------------------
>
>                 Key: KAFKA-1644
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1644
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Anton Karamanov
>            Assignee: Anton Karamanov
>         Attachments: 
> 0001-KAFKA-1644-Inherit-FetchResponse-from-RequestOrRespo.patch
>
>
> Unlike all other Kafka API responses {{FetchResponse}} is not a subclass of 
> RequestOrResponse, which requires handling it as a special case while 
> processing responses.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to