Merge pull request #489 from veho-technologies/fix/metrics-missing-na…

…mespace fix: Metrics not including Namespace
sensedeep · Jul 31, 2023 · 6c67f6c · 6c67f6c
2 parents 330779d + 5aae448
commit 6c67f6c
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/Metrics.js b/src/Metrics.js
@@ -186,19 +186,19 @@ export class Metrics {
 
     emitMetrics(timestamp, rec) {
         let {dimensionValues, dimensions, properties, totals} = rec
-        let metrics = this.metrics
+        let metricsParams = this.metrics
 
         let requests = totals.requests
         totals.latency = totals.latency / requests
         totals.count = totals.count / requests
         totals.scanned = totals.scanned / requests
 
         if (this.log.metrics) {
-            let chan = metrics.chan || 'dbmetrics'
+            let chan = metricsParams.chan || 'dbmetrics'
             this.log.metrics(
                 chan,
                 `OneTable Custom Metrics ${dimensions}`,
-                metrics.namespace,
+                metricsParams.namespace,
                 totals,
                 dimensions,
                 {latency: 'Milliseconds', default: 'Count'},
@@ -215,7 +215,7 @@ export class Metrics {
                         CloudWatchMetrics: [
                             {
                                 Dimensions: [dimensions],
-                                Namespace: metrics.namespace,
+                                Namespace: metricsParams.namespace,
                                 Metrics: metrics,
                             },
                         ],