suneet-s commented on code in PR #12456:
URL: https://github.com/apache/druid/pull/12456#discussion_r853102413


##########
docs/operations/metrics.md:
##########
@@ -58,6 +58,13 @@ Metrics may have additional dimensions beyond those listed 
above.
 |`query/priority`|Assigned lane and priority, only if Laning strategy is 
enabled. Refer to [Laning 
strategies](../configuration/index.md#laning-strategies)|lane, dataSource, 
type|0|
 |`sqlQuery/time`|Milliseconds taken to complete a SQL query.|id, 
nativeQueryIds, dataSource, remoteAddress, success.|< 1s|
 |`sqlQuery/bytes`|Number of bytes returned in the SQL query response.|id, 
nativeQueryIds, dataSource, remoteAddress, success.| |
+|`segment/detailed/numRows`|Number of rows in a segment. Only reported if 
`SchemaStatsMonitor` is enabled.|`dataSource`, `segment/binaryVersion`, 
`segment/hasLastCompactedState`| Varies. Good sized segments should be around 
5M rows.|
+|`segment/detailed/numReplicas`|Number servers that this segment is available 
on. Only reported if `SchemaStatsMonitor` is enabled.|`dataSource`, 
`segment/binaryVersion`, `segment/hasLastCompactedState`| Varies. Should be the 
number of replicas configured.|
+|`segment/detailed/sizeBytes`|Size of the segment in bytes. Only reported if 
`SchemaStatsMonitor` is enabled.|`dataSource`, `segment/binaryVersion`, 
`segment/hasLastCompactedState`| Varies.|
+|`segment/detailed/numDimensions`|Number of dimensions in the segment 
metadata. Only reported if `SchemaStatsMonitor` is enabled.|`dataSource`, 
`segment/binaryVersion`, `segment/hasLastCompactedState`| Varies.|
+|`segment/detailed/numMetrics`|Number of metrics in the segment metadata. Only 
reported if `SchemaStatsMonitor` is enabled.|`dataSource`, 
`segment/binaryVersion`, `segment/hasLastCompactedState`| Varies.|
+|`segment/detailed/durationMillis`|The size of the interval that the segment 
covers in millis. Only reported if `SchemaStatsMonitor` is 
enabled.|`dataSource`, `segment/binaryVersion`, 
`segment/hasLastCompactedState`| Varies.|

Review Comment:
   Good point. How about `segment/detailed/interval/durationMillis`?



-- 
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]

Reply via email to