PR feedback

Author: JamesNK

docs/dotnet/dotnet-aspnetcore-metrics.md

@@ -319,9 +319,9 @@ Meter name: `Microsoft.AspNetCore.RateLimiting`; Added in: ASP.NET Core 8.0
 
 All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` meter.
 
-### Metric: `aspnetcore.memory_pool.current_memory`
+### Metric: `aspnetcore.memory_pool.pooled`
 
-<!-- semconv metric.aspnetcore.memory_pool.current_memory -->
+<!-- semconv metric.aspnetcore.memory_pool.pooled -->
 <!-- NOTE: THIS TEXT IS AUTOGENERATED. DO NOT EDIT BY HAND. -->
 <!-- see templates/registry/markdown/snippet.md.j2 -->
 <!-- prettier-ignore-start -->
@@ -330,7 +330,7 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 
 | Name     | Instrument Type | Unit (UCUM) | Description    | Stability | Entity Associations |
 | -------- | --------------- | ----------- | -------------- | --------- | ------ |
-| `aspnetcore.memory_pool.current_memory` | UpDownCounter | `By` | Number of bytes that are currently pooled by the pool. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
+| `aspnetcore.memory_pool.pooled` | UpDownCounter | `By` | Number of bytes currently pooled and available for reuse. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
 
 **[1]:** Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
 
@@ -345,9 +345,9 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 <!-- END AUTOGENERATED TEXT -->
 <!-- endsemconv -->
 
-### Metric: `aspnetcore.memory_pool.evicted_memory`
+### Metric: `aspnetcore.memory_pool.evicted`
 
-<!-- semconv metric.aspnetcore.memory_pool.evicted_memory -->
+<!-- semconv metric.aspnetcore.memory_pool.evicted -->
 <!-- NOTE: THIS TEXT IS AUTOGENERATED. DO NOT EDIT BY HAND. -->
 <!-- see templates/registry/markdown/snippet.md.j2 -->
 <!-- prettier-ignore-start -->
@@ -356,7 +356,7 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 
 | Name     | Instrument Type | Unit (UCUM) | Description    | Stability | Entity Associations |
 | -------- | --------------- | ----------- | -------------- | --------- | ------ |
-| `aspnetcore.memory_pool.evicted_memory` | Counter | `By` | Total number of bytes that have been evicted. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
+| `aspnetcore.memory_pool.evicted` | Counter | `By` | Total number of bytes evicted from the memory pool. Eviction occurs when idle pooled memory is reclaimed. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
 
 **[1]:** Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
 
@@ -371,9 +371,9 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 <!-- END AUTOGENERATED TEXT -->
 <!-- endsemconv -->
 
-### Metric: `aspnetcore.memory_pool.total_allocated`
+### Metric: `aspnetcore.memory_pool.allocated`
 
-<!-- semconv metric.aspnetcore.memory_pool.total_allocated -->
+<!-- semconv metric.aspnetcore.memory_pool.allocated -->
 <!-- NOTE: THIS TEXT IS AUTOGENERATED. DO NOT EDIT BY HAND. -->
 <!-- see templates/registry/markdown/snippet.md.j2 -->
 <!-- prettier-ignore-start -->
@@ -382,7 +382,7 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 
 | Name     | Instrument Type | Unit (UCUM) | Description    | Stability | Entity Associations |
 | -------- | --------------- | ----------- | -------------- | --------- | ------ |
-| `aspnetcore.memory_pool.total_allocated` | Counter | `By` | Total number of allocations made by the pool. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
+| `aspnetcore.memory_pool.allocated` | Counter | `By` | Total number of bytes allocated by the memory pool. Allocation occurs when a memory rental request exceeds the available pooled memory. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
 
 **[1]:** Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
 
@@ -397,9 +397,9 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 <!-- END AUTOGENERATED TEXT -->
 <!-- endsemconv -->
 
-### Metric: `aspnetcore.memory_pool.total_rented`
+### Metric: `aspnetcore.memory_pool.rented`
 
-<!-- semconv metric.aspnetcore.memory_pool.total_rented -->
+<!-- semconv metric.aspnetcore.memory_pool.rented -->
 <!-- NOTE: THIS TEXT IS AUTOGENERATED. DO NOT EDIT BY HAND. -->
 <!-- see templates/registry/markdown/snippet.md.j2 -->
 <!-- prettier-ignore-start -->
@@ -408,7 +408,7 @@ All memory pool metrics are reported by the `Microsoft.AspNetCore.MemoryPool` me
 
 | Name     | Instrument Type | Unit (UCUM) | Description    | Stability | Entity Associations |
 | -------- | --------------- | ----------- | -------------- | --------- | ------ |
-| `aspnetcore.memory_pool.total_rented` | Counter | `By` | Total number of rented bytes from the pool. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
+| `aspnetcore.memory_pool.rented` | Counter | `By` | Total number of bytes rented from the memory pool. [1] | ![Development](https://img.shields.io/badge/-development-blue) |  |
 
 **[1]:** Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
 

model/aspnetcore/metrics.yaml

@@ -144,56 +144,56 @@ groups:
         requirement_level: required
 
     # memory_pool
-  - id: metric.aspnetcore.memory_pool.current_memory
+  - id: metric.aspnetcore.memory_pool.pooled
     type: metric
-    metric_name: aspnetcore.memory_pool.current_memory
+    metric_name: aspnetcore.memory_pool.pooled
     annotations:
       code_generation:
         metric_value_type: int
     stability: development
-    brief: Number of bytes that are currently pooled by the pool.
+    brief: Number of bytes currently pooled and available for reuse.
     instrument: updowncounter
     unit: "By"
     note: |
       Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
     extends: aspnetcore.common.memory_pool.metrics.attributes
 
-  - id: metric.aspnetcore.memory_pool.total_allocated
+  - id: metric.aspnetcore.memory_pool.allocated
     type: metric
-    metric_name: aspnetcore.memory_pool.total_allocated
+    metric_name: aspnetcore.memory_pool.allocated
     annotations:
       code_generation:
         metric_value_type: int
     stability: development
-    brief: Total number of allocations made by the pool.
+    brief: Total number of bytes allocated by the memory pool. Allocation occurs when a memory rental request exceeds the available pooled memory.
     instrument: counter
     unit: "By"
     note: |
       Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
     extends: aspnetcore.common.memory_pool.metrics.attributes
 
-  - id: metric.aspnetcore.memory_pool.evicted_memory
+  - id: metric.aspnetcore.memory_pool.evicted
     type: metric
-    metric_name: aspnetcore.memory_pool.evicted_memory
+    metric_name: aspnetcore.memory_pool.evicted
     annotations:
       code_generation:
         metric_value_type: int
     stability: development
-    brief: Total number of bytes that have been evicted.
+    brief: Total number of bytes evicted from the memory pool. Eviction occurs when idle pooled memory is reclaimed.
     instrument: counter
     unit: "By"
     note: |
       Meter name: `Microsoft.AspNetCore.MemoryPool`; Added in: ASP.NET Core 10.0
     extends: aspnetcore.common.memory_pool.metrics.attributes
 
-  - id: metric.aspnetcore.memory_pool.total_rented
+  - id: metric.aspnetcore.memory_pool.rented
     type: metric
-    metric_name: aspnetcore.memory_pool.total_rented
+    metric_name: aspnetcore.memory_pool.rented
     annotations:
       code_generation:
         metric_value_type: int
     stability: development
-    brief: Total number of rented bytes from the pool.
+    brief: Total number of bytes rented from the memory pool.
     instrument: counter
     unit: "By"
     note: |