Mercurial > hg > thermostat-ng > web-client

changeset 271:671cfc1d001d

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Streamline jvm-memory metric conversion filters Reviewed-by: jkang Review-thread: http://icedtea.classpath.org/pipermail/thermostat/2017-October/025371.html
author Andrew Azores <aazores@redhat.com>
date Wed, 25 Oct 2017 11:51:03 -0400
parents 4023b79e4d58
children b35a283eb016
files src/app/components/jvm-info/jvm-memory/jvm-memory.controller.js
diffstat 1 files changed, 3 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/src/app/components/jvm-info/jvm-memory/jvm-memory.controller.js	Wed Oct 11 14:59:09 2017 -0400
+++ b/src/app/components/jvm-info/jvm-memory/jvm-memory.controller.js	Wed Oct 25 11:51:03 2017 -0400
@@ -32,10 +32,8 @@
 import service from './jvm-memory.service.js';
 
 class JvmMemoryController {
-  constructor ($stateParams, $interval, jvmMemoryService, metricToBigIntFilter,
-    bigIntToStringFilter, stringToNumberFilter, dateFilter, DATE_FORMAT,
-    metricToNumberFilter, scaleBytesService,
-    sanitizeService, $translate) {
+  constructor ($stateParams, $interval, jvmMemoryService, dateFilter,
+    DATE_FORMAT, metricToNumberFilter, scaleBytesService, sanitizeService, $translate) {
     'ngInject';
 
     this.jvmId = $stateParams.jvmId;
@@ -44,9 +42,6 @@
     this._sanitizeService = sanitizeService;
     this._translate = $translate;
 
-    this._metricToBigInt = metricToBigIntFilter;
-    this._bigIntToString = bigIntToStringFilter;
-    this._stringToNumber = stringToNumberFilter;
     this._dateFilter = dateFilter;
     this._dateFormat = DATE_FORMAT;
     this._metricToNumber = metricToNumberFilter;
@@ -385,10 +380,7 @@
   }
 
   convertMemStat (stat, scale) {
-    let bigInt = this._metricToBigInt(stat, scale);
-    let str = this._bigIntToString(bigInt);
-    let num = this._stringToNumber(str);
-    return _.ceil(num);
+    return _.ceil(this._metricToNumber(stat, scale));
   }
 
   sanitize (str) {