Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

Add more runtime.Memstats to reported metrics #183

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

prateek
Copy link
Contributor

@prateek prateek commented Jul 3, 2018

Pulls in a few more https://golang.org/pkg/runtime/#MemStats fields:

  • Lookups
  • NextGC
  • HeapObjects

@prateek prateek requested review from jeromefroe, robskillington and xichen2020 and removed request for robskillington July 3, 2018 00:06
@codecov
Copy link

codecov bot commented Jul 3, 2018

Codecov Report

Merging #183 into master will decrease coverage by 0.55%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #183      +/-   ##
==========================================
- Coverage   78.62%   78.06%   -0.56%     
==========================================
  Files          86       86              
  Lines        3518     3524       +6     
==========================================
- Hits         2766     2751      -15     
- Misses        684      705      +21     
  Partials       68       68

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 647a4a0...947accc. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant