jihoonson commented on a change in pull request #11394:
URL: https://github.com/apache/druid/pull/11394#discussion_r670708356
##########
File path: docs/operations/metrics.md
##########
@@ -59,7 +59,7 @@ Available Metrics
|`query/failed/count`|number of failed queries|This metric is only available
if the QueryCountStatsMonitor module is included.||
|`query/interrupted/count`|number of queries interrupted due to
cancellation.|This metric is only available if the QueryCountStatsMonitor
module is included.||
|`query/timeout/count`|number of timed out queries.|This metric is only
available if the QueryCountStatsMonitor module is included.||
-|`query/segments/count`|This query is not enabled by default. See the
`QueryMetrics` Interface for reference regarding enabling this metric. Number
of segments that will be touched by the query.|Varies.|
+|`query/segments/count`|This query is not enabled by default. See the
`QueryMetrics` Interface for reference regarding enabling this metric. Number
of segments that will be touched by the query. If the query has to be retried,
the metric will be reported for all retries as well as the original
query.|Varies.|
Review comment:
I tried to add some more details. Please feel free to edit it if you
like.
```
In the broker, it makes a plan to distribute the query to realtime tasks and
historicals based on a snapshot of segment distribution state. If there are
some segments moved after this snapshot is created, certain historicals and
realtime tasks can report those segments as missing to the broker. The broker
will re-send the query to the new servers that serve those segments after move.
In this case, those segments can be counted more than once in this metric.
```
--
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]
