MINOR: Cleanup toString methods in Storage Module (#20432)

Getting rid of a bunch of `toString` functions in record classes in
Storage Module.

Reviewers: Chia-Ping Tsai <[email protected]>

Author: sjhajharia

storage/src/main/java/org/apache/kafka/storage/internals/log/EpochEntry.java

@@ -18,12 +18,4 @@
 
 // Mapping of epoch to the first offset of the subsequent epoch
 public record EpochEntry(int epoch, long startOffset) {
-
-    @Override
-    public String toString() {
-        return "EpochEntry(" +
-                "epoch=" + epoch +
-                ", startOffset=" + startOffset +
-                ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/LastRecord.java

@@ -27,12 +27,4 @@ public record LastRecord(OptionalLong lastDataOffset, short producerEpoch) {
     public LastRecord {
         Objects.requireNonNull(lastDataOffset, "lastDataOffset must be non null");
     }
-
-    @Override
-    public String toString() {
-        return "LastRecord(" +
-                "lastDataOffset=" + lastDataOffset +
-                ", producerEpoch=" + producerEpoch +
-                ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/LoadedLogOffsets.java

@@ -26,13 +26,4 @@ public LoadedLogOffsets(final long logStartOffset,
         this.recoveryPoint = recoveryPoint;
         this.nextOffsetMetadata = Objects.requireNonNull(nextOffsetMetadata, "nextOffsetMetadata should not be null");
     }
-
-    @Override
-    public String toString() {
-        return "LoadedLogOffsets(" +
-                "logStartOffset=" + logStartOffset +
-                ", recoveryPoint=" + recoveryPoint +
-                ", nextOffsetMetadata=" + nextOffsetMetadata +
-                ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/LogOffsetSnapshot.java

@@ -23,14 +23,4 @@
  */
 public record LogOffsetSnapshot(long logStartOffset, LogOffsetMetadata logEndOffset, LogOffsetMetadata highWatermark,
                                 LogOffsetMetadata lastStableOffset) {
-
-    @Override
-    public String toString() {
-        return "LogOffsetSnapshot(" +
-                "logStartOffset=" + logStartOffset +
-                ", logEndOffset=" + logEndOffset +
-                ", highWatermark=" + highWatermark +
-                ", lastStableOffset=" + lastStableOffset +
-                ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/LogReadInfo.java

@@ -25,16 +25,4 @@
  */
 public record LogReadInfo(FetchDataInfo fetchedData, Optional<FetchResponseData.EpochEndOffset> divergingEpoch,
                           long highWatermark, long logStartOffset, long logEndOffset, long lastStableOffset) {
-
-    @Override
-    public String toString() {
-        return "LogReadInfo(" +
-                "fetchedData=" + fetchedData +
-                ", divergingEpoch=" + divergingEpoch +
-                ", highWatermark=" + highWatermark +
-                ", logStartOffset=" + logStartOffset +
-                ", logEndOffset=" + logEndOffset +
-                ", lastStableOffset=" + lastStableOffset +
-                ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/OffsetPosition.java

@@ -32,12 +32,4 @@ public long indexKey() {
     public long indexValue() {
         return position;
     }
-
-    @Override
-    public String toString() {
-        return "OffsetPosition(" +
-            "offset=" + offset +
-            ", position=" + position +
-            ')';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/RemoteStorageFetchInfo.java

@@ -22,15 +22,4 @@
 
 public record RemoteStorageFetchInfo(int fetchMaxBytes, boolean minOneMessage, TopicIdPartition topicIdPartition,
                                      FetchRequest.PartitionData fetchInfo, FetchIsolation fetchIsolation) {
-
-    @Override
-    public String toString() {
-        return "RemoteStorageFetchInfo{" +
-                "fetchMaxBytes=" + fetchMaxBytes +
-                ", minOneMessage=" + minOneMessage +
-                ", topicIdPartition=" + topicIdPartition +
-                ", fetchInfo=" + fetchInfo +
-                ", fetchIsolation=" + fetchIsolation +
-                '}';
-    }
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/RollParams.java

@@ -21,17 +21,4 @@
  */
 public record RollParams(long maxSegmentMs, int maxSegmentBytes, long maxTimestampInMessages, long maxOffsetInMessages,
                          int messagesSize, long now) {
-
-    @Override
-    public String toString() {
-        return "RollParams(" +
-                "maxSegmentMs=" + maxSegmentMs +
-                ", maxSegmentBytes=" + maxSegmentBytes +
-                ", maxTimestampInMessages=" + maxTimestampInMessages +
-                ", maxOffsetInMessages=" + maxOffsetInMessages +
-                ", messagesSize=" + messagesSize +
-                ", now=" + now +
-                ')';
-    }
-
 }

storage/src/main/java/org/apache/kafka/storage/internals/log/TimestampOffset.java

@@ -36,11 +36,4 @@ public long indexKey() {
     public long indexValue() {
         return offset;
     }
-
-    @Override
-    public String toString() {
-        return String.format("TimestampOffset(offset = %d, timestamp = %d)",
-            offset,
-            timestamp);
-    }
 }