kpatel006 commented on code in PR #19891:
URL: https://github.com/apache/pinot/pull/19891#discussion_r4420255375
##########
pinot-common/src/main/java/org/apache/pinot/common/metrics/BrokerMeter.java:
##########
@@ -216,6 +216,21 @@ public class BrokerMeter implements AbstractMetrics.Meter {
public static final BrokerMeter ROUTING_TABLE_UPDATE_FAILURE =
create("ROUTING_TABLE_UPDATE_FAILURE", "failures", false);
+ /// Number of queries that touched a segment eligible for fail-open routing
(unavailable, but with candidate servers
+ /// that are all disabled). Counted once per query per physical table, even
if the query routes that table more than
+ /// once (e.g. a multi-stage self-join); concurrent queries routing the same
table in between can rarely recount it.
+ ///
+ /// This metric is global.
+ public static final BrokerMeter FAIL_OPEN_ELIGIBLE_REQUESTS =
Review Comment:
`FAIL_OPEN_ELIGIBLE_REQUESTS ` was meant more as a global meter to help
determine if fail-open should be enabled or not. The reason that its a global
meter is because fail-open would be enabled for the whole broker and not per
table.
`FAIL_OPEN_ELIGIBLE_SEGMENTS` was meant to show which tables were actually
failing open.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
