timsants commented on a change in pull request #8358:
URL: https://github.com/apache/pinot/pull/8358#discussion_r834762713
##########
File path:
pinot-controller/src/main/java/org/apache/pinot/controller/helix/SegmentStatusChecker.java
##########
@@ -57,21 +62,30 @@
private static final long DISABLED_TABLE_LOG_INTERVAL_MS =
TimeUnit.DAYS.toMillis(1);
private static final ZNRecordSerializer RECORD_SERIALIZER = new
ZNRecordSerializer();
+ private static final int TABLE_CHECKER_TIMEOUT_MS = 30_000;
+
private final int _waitForPushTimeSeconds;
private long _lastDisabledTableLogTimestamp = 0;
+ private TableSizeReader _tableSizeReader;
+ private ExecutorService _executorService;
Review comment:
I removed `_executorService` but kept `_tableSizeReader` as is because I
set it with a mocked object it in unit tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
