XMLWordPrintableJSON

    Details

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

      Description

      We need an important fix for handling the user mis-configuration for duplicate group.instance.ids. Several approaches we have discussed so far:

      1. Limit resetGeneration() call to only JoinGroupResponseHandler
      2. Include InstanceId in the Heartbeat and OffsetCommit APIs. Then the coordinator can return the proper error code.
      3. We can can use a convention to embed the instanceId into the generated memberId. At the moment, the current format is {{ {clientId}

        -{random uuid}}}. For static members, I think instanceId is more useful than clientId and we could probably use timestamp as a more concise alternative to uuid. So we could have {{

        {instanceId}

        -{timestamp}}} as the memberId for static members. Then we would be able to extract this from any request and the coordinator could use the proper error code

      Right now we are more inclined to option 2 or 3, however it requires non-trivial amount of code changes including protocol changes and fatal error handling on client side. 

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                bchen225242 Boyang Chen
                Reporter:
                bchen225242 Boyang Chen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: