This is an automated email from the ASF dual-hosted git repository.
xiangfu0 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new ade41d16899 Log lead-controller VIP lookup at debug, not info (#19758)
ade41d16899 is described below
commit ade41d16899eeb0ca46d08600e2d598fbbf795a6
Author: shivam-startree <[email protected]>
AuthorDate: Tue Oct 6 16:02:16 2026 +0530
Log lead-controller VIP lookup at debug, not info (#19758)
ClusterInfoAccessor.getVipUrlForLeadController() logs at INFO every time
the current controller is the table's lead. Minion task generators call it
at least once per generated task (for the upload URI and the controller
URI), so a generator that plans many tasks floods the controller log. On a
production cluster, planning refresh tasks for ~175k segments of one table
wrote ~490k copies of this line in about three minutes, on every run.
The non-leader success path already returns without logging, and the two
fallback paths keep their WARN, so this only quiets the normal case.
Co-authored-by: Claude Opus 5.5 <[email protected]>
---
.../apache/pinot/controller/helix/core/minion/ClusterInfoAccessor.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/ClusterInfoAccessor.java
b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/ClusterInfoAccessor.java
index 6e0dc9c19c0..8fb20cd7694 100644
---
a/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/ClusterInfoAccessor.java
+++
b/pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/minion/ClusterInfoAccessor.java
@@ -182,7 +182,7 @@ public class ClusterInfoAccessor {
public String getVipUrlForLeadController(String tableNameWithType) {
String rawTableName =
TableNameBuilder.extractRawTableName(tableNameWithType);
if (_leadControllerManager.isLeaderForTable(rawTableName)) {
- LOGGER.info("Controller is leader for table {}", tableNameWithType);
+ LOGGER.debug("Controller is leader for table {}", tableNameWithType);
return getVipUrl();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
