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

Joe Stein commented on KAFKA-240:
---------------------------------

It is written to the buffer though it is not passed in when constructing the 
ProducerRequest

val version_id: Short = ProducerRequest.version_id

happens during the constructor

since actual changes come in the code I though having this be on the object 
leve and part of the jar the publisher was calling made the most sense.  It 
seemed that the client might not need or want to have this knowledge but I can 
pass it into the constructor, not a problem.

Also Prashanth pointed out my member vars are not consistent with the rest of 
the system and camel case I will make that change also to be consistent.
                
> implement new producer and consumer request format
> --------------------------------------------------
>
>                 Key: KAFKA-240
>                 URL: https://issues.apache.org/jira/browse/KAFKA-240
>             Project: Kafka
>          Issue Type: Sub-task
>          Components: core
>    Affects Versions: 0.8
>            Reporter: Jun Rao
>              Labels: fetch, replication, wireprotocol
>             Fix For: 0.8
>
>         Attachments: KAFKA-240-FetchRequest-v1.patch, 
> KAFKA-240-FetchRequest-v2.patch, KAFKA-240-FetchRequest-validation-v1.patch, 
> KAFKA-240.ProducerRequest.v2.patch, KAFKA-240.ProducerRequest.v3.patch, 
> KAFKA-240.ProducerRequest.v4.patch, KAFKA-240.v3.patch, 
> kafka-240_unittestfix_delta.patch
>
>
> We want to change the producer/consumer request/response format according to 
> the discussion in the following wiki:
> https://cwiki.apache.org/confluence/display/KAFKA/New+Wire+Format+Proposal

--
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