Open fernyettheplant opened 5 months ago
Thanks for filing this issue, @fernyettheplant. @kushagraThapar could you route this?
@fernyettheplant thanks for raising this issue, @FabianMeiswinkel can you please take a look at this and guide us in the right direction. How should we fix this issue?
Is your feature request related to a problem? Please describe. Our team is currently using CosmosDB in our Java apps and we are integrating Dynatrace as our monitoring and alerting tool. However, after looking into an issue where we cannot see the CosmosDB metrics coming from the app and a warning related to the unit
RU (Request Unit)
coming from this line of code from Dynatrace we found out that the unit is hardcoded on these lineshttps://github.com/Azure/azure-sdk-for-java/blob/f94f88cd167d86a190200477e7c67137b1bd09c3/sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/clienttelemetry/ClientTelemetryMetrics.java#L450
https://github.com/Azure/azure-sdk-for-java/blob/f94f88cd167d86a190200477e7c67137b1bd09c3/sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/clienttelemetry/ClientTelemetryMetrics.java#L945
https://github.com/Azure/azure-sdk-for-java/blob/f94f88cd167d86a190200477e7c67137b1bd09c3/sdk/cosmos/azure-cosmos/src/main/java/com/azure/cosmos/implementation/clienttelemetry/ClientTelemetryMetrics.java#L1073
Describe the solution you'd like A way to customize the Units that are going to be sent to Micrometer to be flexible with any vendor
Describe alternatives you've considered Make a PR to change only those three lines to just
RU
orRequest_Unit
but that may impact the metrics of folks that are already using the metrics emitted by the SDKAdditional context Nope
Information Checklist Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report
Thank you in advance 🙂