Kafka
  1. Kafka
  2. KAFKA-617

Refactor KafkaApis.handleOffsetRequest

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      This code path is just really bad.

      All the business logic is in LogManager and Log.

      Also the implementation is really funky.

      LogManager already supports getLog and log allows you to get the segment list so there shouldn't be any logic except in KafkaApis.

        Activity

          People

          • Assignee:
            Unassigned
            Reporter:
            Jay Kreps
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development