Kafka
  1. Kafka
  2. KAFKA-316

disallow recursively compressed message

    Details

    • Type: Improvement Improvement
    • Status: Patch Available
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: core
    • Labels:
      None

      Description

      Currently, it is possible to create a compressed Message that contains a set of Messages, each of which is further compressed. Support recursively compressed messages has little benefit and can complicates the on disk storage format. We should probably disallow this.

        Activity

        Jun Rao created issue -
        Joe Stein made changes -
        Field Original Value New Value
        Fix Version/s 0.8 [ 12317244 ]
        Fix Version/s 0.7.1 [ 12319140 ]
        David Arthur made changes -
        Hide
        David Arthur added a comment -

        Add a "depth" variable to keep track of recursion level. If depth > 0, throw an InvalidMessageException

        Show
        David Arthur added a comment - Add a "depth" variable to keep track of recursion level. If depth > 0, throw an InvalidMessageException
        David Arthur made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Joe Stein made changes -
        Fix Version/s 0.8.1 [ 12322960 ]
        Fix Version/s 0.8 [ 12317244 ]
        Hide
        Sriram Subramanian added a comment -

        I don't see any reason to prevent nested messages. Do we need to fix this?

        Show
        Sriram Subramanian added a comment - I don't see any reason to prevent nested messages. Do we need to fix this?
        Hide
        David Arthur added a comment -

        In addition to the reasons citing in the description, it makes client message decoding more complex. I'm not sure what reason there would be to keep it.

        Show
        David Arthur added a comment - In addition to the reasons citing in the description, it makes client message decoding more complex. I'm not sure what reason there would be to keep it.
        Neha Narkhede made changes -
        Fix Version/s 0.9.0 [ 12323928 ]
        Fix Version/s 0.8.1 [ 12322960 ]
        Tony Stevenson made changes -
        Workflow no-reopen-closed, patch-avail [ 12659424 ] Apache Kafka Workflow [ 13052937 ]
        Tony Stevenson made changes -
        Workflow Apache Kafka Workflow [ 13052937 ] no-reopen-closed, patch-avail [ 13055512 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        374d 15h 1m 1 David Arthur 02/Apr/13 15:22

          People

          • Assignee:
            Unassigned
            Reporter:
            Jun Rao
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development