Don't log segment name in TimeSegmentPruner (#14855)

apache · Jan 21, 2025 · 2babb6f · 2babb6f
1 parent 085c995
commit 2babb6f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/...broker/src/main/java/org/apache/pinot/broker/routing/segmentpruner/TimeSegmentPruner.java b/...broker/src/main/java/org/apache/pinot/broker/routing/segmentpruner/TimeSegmentPruner.java
@@ -93,10 +93,11 @@ private Interval extractIntervalFromSegmentZKMetaZNRecord(String segment, @Nulla
       return DEFAULT_INTERVAL;
     }
 
+    // Validate time interval
     long startTime = znRecord.getLongField(CommonConstants.Segment.START_TIME, -1);
     long endTime = znRecord.getLongField(CommonConstants.Segment.END_TIME, -1);
     if (startTime < 0 || endTime < 0 || startTime > endTime) {
-      LOGGER.warn("Failed to find valid time interval for segment: {}, table: {}", segment, _tableNameWithType);
+      // Consuming and committing segments don't have time interval
       return DEFAULT_INTERVAL;
     }