Skip to content

Commit 1553498

Browse files
committed
Improve metric descriptions
1 parent cb7eb48 commit 1553498

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

quickwit/quickwit-storage/src/metrics.rs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,22 +78,22 @@ impl Default for StorageMetrics {
7878
object_storage_request_duration: new_histogram_vec(
7979
"object_storage_request_duration",
8080
"Durations until the response headers are returned from the object store, by \
81-
action and status. This does not measure the download time.",
81+
action and status. This does not measure the download time for the body content.",
8282
"storage",
8383
&[],
8484
["action", "status"],
8585
vec![0.1, 0.5, 1.0, 5.0, 10.0, 30.0, 60.0],
8686
),
8787
object_storage_get_slice_in_flight_count: new_gauge(
8888
"object_storage_get_slice_in_flight_count",
89-
"Number of GetObject for which the memory was allocated but the download is still \
90-
in progress.",
89+
"Number of get_object for which the memory was allocated but the download is \
90+
still in progress.",
9191
"storage",
9292
&[],
9393
),
9494
object_storage_get_slice_in_flight_num_bytes: new_gauge(
9595
"object_storage_get_slice_in_flight_num_bytes",
96-
"Memory allocated for GetObject requests that are still in progress.",
96+
"Memory allocated for get_object requests that are still in progress.",
9797
"storage",
9898
&[],
9999
),
@@ -106,6 +106,9 @@ impl Default for StorageMetrics {
106106
),
107107
object_storage_download_errors: new_counter_vec(
108108
"object_storage_download_errors",
109+
// Download errors are recorded separately because the associated
110+
// get_object requests were already recorded as successful in
111+
// object_storage_requests_total
109112
"Number of download requests that received successful response headers but failed \
110113
during download.",
111114
"storage",

0 commit comments

Comments
 (0)