Kafka
  1. Kafka
  2. KAFKA-1326 New consumer checklist
  3. KAFKA-1462

Add new request and response formats for the new consumer and coordinator communication

    Details

    • Type: Sub-task Sub-task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.8.2.0
    • Component/s: consumer
    • Labels:
      None

      Description

      We need to add the request / response formats according to the new format protocol once their design is final:

      https://cwiki.apache.org/confluence/display/KAFKA
      /Kafka+0.9+Consumer+Rewrite+Design

      1. KAFKA-1462.patch
        16 kB
        Jun Rao
      2. KAFKA-1462_2014-07-16_21:39:07.patch
        183 kB
        Jun Rao
      3. KAFKA-1462.patch
        180 kB
        Jun Rao

        Issue Links

          Activity

          Hide
          Jun Rao added a comment -

          Created reviewboard https://reviews.apache.org/r/23516/
          against branch origin/trunk

          Show
          Jun Rao added a comment - Created reviewboard https://reviews.apache.org/r/23516/ against branch origin/trunk
          Hide
          Jun Rao added a comment -

          Updated reviewboard https://reviews.apache.org/r/23516/
          against branch origin/trunk

          Show
          Jun Rao added a comment - Updated reviewboard https://reviews.apache.org/r/23516/ against branch origin/trunk
          Hide
          Jun Rao added a comment -

          Thanks for the reviews. Committed to trunk.

          Show
          Jun Rao added a comment - Thanks for the reviews. Committed to trunk.
          Hide
          Jun Rao added a comment -

          There is actually an issue with the patch. The server side objects can't use the full request header right now since it doesn't include the request id. Will submit a followup patch to address this.

          Show
          Jun Rao added a comment - There is actually an issue with the patch. The server side objects can't use the full request header right now since it doesn't include the request id. Will submit a followup patch to address this.
          Hide
          Jun Rao added a comment -

          Created reviewboard https://reviews.apache.org/r/23740/
          against branch origin/trunk

          Show
          Jun Rao added a comment - Created reviewboard https://reviews.apache.org/r/23740/ against branch origin/trunk
          Hide
          Jun Rao added a comment -

          Thanks for the reviews. Committed the followup patch.

          Show
          Jun Rao added a comment - Thanks for the reviews. Committed the followup patch.

            People

            • Assignee:
              Jun Rao
              Reporter:
              Guozhang Wang
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development