Uploaded image for project: 'Kafka'
  1. Kafka
  2. KAFKA-15866

Refactor OffsetFetchRequestState Error handling to be more consistent with OffsetCommitRequestState

    XMLWordPrintableJSON

Details

    Description

      The current OffsetFetchRequestState error handling uses nested if-else, which is quite different, stylistically, to the OffsetCommitRequestState using a switch statment.  The latter is a bit more readable so we should refactor the error handling using the same style to improve readability.

       

      A minor point: Some of the error handling seems inconsistent with the commit. The logic was from the current implementation, so we should also review all the error handling.  For example: somehow the current logic doesn't mark the coordinator unavailable when receiving COORDINATOR_NOT_AVAILABLE

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              pnee Philip Nee
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: