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

Refactor KafkaApis.handleListOffsetRequest

    Details

    • Type: Improvement
    • Status: Open
    • Priority: 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.

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated: