[APM] fixes incorrect values in service overview throughput chart (#8…

…9348) * [APM] fixes incorrect values in service overview throughput chart
elastic · Jan 29, 2021 · 8e57b63 · 8e57b63
1 parent 7465976
commit 8e57b63
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 32 deletions.
diff --git a/x-pack/plugins/apm/server/lib/services/get_throughput.ts b/x-pack/plugins/apm/server/lib/services/get_throughput.ts
@@ -27,12 +27,17 @@ interface Options {
 
 type ESResponse = PromiseReturnType<typeof fetcher>;
 
-function transform(response: ESResponse) {
+function transform(response: ESResponse, options: Options) {
+  const { end, start } = options.setup;
+  const deltaAsMinutes = (end - start) / 1000 / 60;
   if (response.hits.total.value === 0) {
     return [];
   }
   const buckets = response.aggregations?.throughput.buckets ?? [];
-  return buckets.map(({ key: x, doc_count: y }) => ({ x, y }));
+  return buckets.map(({ key: x, doc_count: y }) => ({
+    x,
+    y: y / deltaAsMinutes,
+  }));
 }
 
 async function fetcher({
@@ -82,6 +87,6 @@ async function fetcher({
 
 export async function getThroughput(options: Options) {
   return {
-    throughput: transform(await fetcher(options)),
+    throughput: transform(await fetcher(options), options),
   };
 }
diff --git a/x-pack/test/apm_api_integration/tests/services/__snapshots__/throughput.snap b/x-pack/test/apm_api_integration/tests/services/__snapshots__/throughput.snap