[GitHub] [skywalking] wu-sheng commented on a diff in pull request #10420: Add cpm5d aggregation function

["wu-sheng (via GitHub)" <gi...@apache.org>]

wu-sheng commented on code in PR #10420:
URL: https://github.com/apache/skywalking/pull/10420#discussion_r1128821193


##########
oap-server/server-starter/src/main/resources/oal/core.oal:
##########
@@ -19,15 +19,15 @@
 // Service scope metrics
 service_resp_time = from(Service.latency).longAvg();
 service_sla = from(Service.*).percent(status == true);
-service_cpm = from(Service.*).cpm();
+service_cpm = from(Service.*).cpm5d();

Review Comment:
   ```suggestion
   service_cpm5d = from(Service.*).cpm5d();
   ```
   
   Let's follow this to update settings. @wankai123 Once this is merged, Grafana setup should be updated according on showcase.



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