Skip to content

Commit b1f4b81

Browse files
committed
refactor(metrics): use documented node-apis
Use `process.memoryUsage()` instead of `v8.getHeapStatistics()`
1 parent db09598 commit b1f4b81

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

lib/metrics/runtime.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
'use strict'
22

3-
const v8 = require('v8')
4-
53
const eventLoopMonitor = require('monitor-event-loop-delay')
64

75
const activeHandles = typeof process._getActiveHandles === 'function'
@@ -43,10 +41,10 @@ class RuntimeCollector {
4341
this.stats['nodejs.eventloop.delay.avg.ms'] = loopDelay
4442
this.loopMonitor.reset()
4543

46-
// Heap
47-
const heap = v8.getHeapStatistics()
48-
this.stats['nodejs.memory.heap.allocated.bytes'] = heap.total_heap_size
49-
this.stats['nodejs.memory.heap.used.bytes'] = heap.used_heap_size
44+
// Memory / Heap
45+
const memoryUsage = process.memoryUsage()
46+
this.stats['nodejs.memory.heap.allocated.bytes'] = memoryUsage.heapTotal
47+
this.stats['nodejs.memory.heap.used.bytes'] = memoryUsage.heapUsed
5048

5149
if (cb) process.nextTick(cb)
5250
}

0 commit comments

Comments
 (0)