Kafka
  1. Kafka
  2. KAFKA-566

Add last modified time to the TopicMetadataRequest

    Details

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

      Description

      To support KAFKA-560 it would be nice to have a last modified time in the TopicMetadataRequest. This would be the timestamp of the last append to the log as taken from stat on the final log segment.

      Implementation would involve
      1. Adding a new field to TopicMetadataRequest
      2. Adding a method Log.lastModified: Long to get the last modified time from a log

      This timestamp would, of course, be subject to error in the event that the file was touched without modification, but I think that is actually okay since it provides a manual way to avoid gc'ing a topic that you know you will want.

      It is debatable whether this should go in 0.8. It would be nice to add the field to the metadata request, at least, as that change should be easy and would avoid needing to bump the version in the future.

        Issue Links

          Activity

          Neha Narkhede made changes -
          Reviewer Neha Narkhede [ nehanarkhede ]
          Sriharsha Chintalapani made changes -
          Assignee Sriharsha Chintalapani [ sriharsha ]
          Chris Riccomini made changes -
          Link This issue is depended upon by KAFKA-560 [ KAFKA-560 ]
          Guozhang Wang made changes -
          Fix Version/s 0.9.0 [ 12323928 ]
          Fix Version/s 0.8.2 [ 12326167 ]
          Neha Narkhede made changes -
          Fix Version/s 0.8.2 [ 12326167 ]
          Fix Version/s 0.8.1 [ 12322960 ]
          Jun Rao made changes -
          Field Original Value New Value
          Fix Version/s 0.8.1 [ 12322960 ]
          Fix Version/s 0.8 [ 12317244 ]
          Priority Blocker [ 1 ] Major [ 3 ]
          Jay Kreps created issue -

            People

            • Assignee:
              Sriharsha Chintalapani
              Reporter:
              Jay Kreps
              Reviewer:
              Neha Narkhede
            • Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

              • Created:
                Updated:

                Development