dmyang commented on code in PR #9697:
URL: https://github.com/apache/pinot/pull/9697#discussion_r1010913789
##########
pinot-core/src/main/java/org/apache/pinot/core/transport/server/routing/stats/ServerRoutingStatsManager.java:
##########
@@ -47,8 +48,13 @@ public class ServerRoutingStatsManager {
private final PinotConfiguration _config;
private volatile boolean _isEnabled;
private ConcurrentHashMap<String, ServerRoutingStatsEntry>
_serverQueryStatsMap;
+
+ // Main executor service for collecting and aggregating stats for all
servers.
private ExecutorService _executorService;
+ // ScheduledExecutorServer for processing periodic tasks like decay.
+ private ScheduledExecutorService _periodicTaskExecutor;
Review Comment:
👍 Thanks for explaining reasoning. That makes sense to me and seems fine to
leave as is.
--
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
