Skip to content

Commit 385df5b

Browse files
committed
Code review.
1 parent ebf9bef commit 385df5b

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

docs/core/diagnostics/metrics-instrumentation.md

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -580,10 +580,11 @@ version `1.10.0`.
580580

581581
> [!IMPORTANT]
582582
> In general more buckets will lead to more precise data for a given Histogram
583-
> but each bucket requires memory to store the aggregated details. It is
584-
> important to understand this tradeoff between precision and memory consumption
585-
> when choosing the number of buckets to recommend via the `InstrumentAdvice`
586-
> API.
583+
> but each bucket requires memory to store the aggregated details and there is
584+
> CPU cost to find the correct bucket when processing a measurement. It is
585+
> important to understand the tradeoffs between precision and CPU/memory
586+
> consumption when choosing the number of buckets to recommend via the
587+
> `InstrumentAdvice` API.
587588
588589
The following code shows an example using the `InstrumentAdvice` API to set
589590
recommended default buckets.

0 commit comments

Comments
 (0)