Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add value field #31

Open
dhopkalo opened this issue Sep 24, 2016 · 1 comment
Open

Add value field #31

dhopkalo opened this issue Sep 24, 2016 · 1 comment
Assignees
Milestone

Comments

@dhopkalo
Copy link
Contributor

dhopkalo commented Sep 24, 2016

Hello,

meminfo_info_dump lacks the value field to generate some useful reports, pls merge my pull request #29 (it contains the fix and tests from #30 issue) because without this fix we can't gather reports in a row. It takes ~1000mb memory to get at least ~30 reports for the php app when it usually takes ~ 100mb of memory for its objects.

Thanks.

@dhopkalo dhopkalo changed the title - add value field Add value field Sep 24, 2016
@BitOne
Copy link
Owner

BitOne commented Nov 17, 2017

That would definitely be a nice addition. As I commented on the PR, it should be an option. I'm closing the PR because it has some problems (doesn't work without JSON extension) and because the code changed a lot.

But I let this issue opened, as I found myself needed this feature in the past ;)

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

No branches or pull requests

2 participants