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