search

status-im / nim-metrics

Nim metrics client library supporting the Prometheus monitoring toolkit, StatsD and Carbon
Other
40 stars 6 forks source link

GC statistics are not in the right thread #38

Closed Menduist closed 2 years ago

Menduist commented 3 years ago

getOccupiedMem & getTotalMem are thread-specific, and since nim-metrics is in his own thread, the nim_gc_mem_bytes & nim_gc_mem_occupied_bytes are not really relevant.

Fortunately, dumpHeapInstances is global, so no issues there

stefantalpalaru commented 3 years ago

Yes, known issue. Just have to get around to fixing it.