[GitHub] [skywalking] kezhenxu94 commented on pull request #10420: Add cpm5d aggregation function

["kezhenxu94 (via GitHub)" <gi...@apache.org>]

kezhenxu94 commented on PR #10420:
URL: https://github.com/apache/skywalking/pull/10420#issuecomment-1459088439

   I have a doubt, now we simply replace the original `cpm` to `cpm5d`, and keep the original metrics name, when users upgrade the OAP but not UI (yet), they will find the related metrics get 100000 times by the real value, this can bring confusion. Also, this is not considered a breaking change, and the metrics data before upgrading and after upgrading will mess up, some of them are original `cpm` and some of them are `x 100000`, the final data make no sense


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@skywalking.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org