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

Implement SHOW STATS FOR 'indexes' #9822

Merged
merged 3 commits into from May 10, 2018
Merged

Implement SHOW STATS FOR 'indexes' #9822

merged 3 commits into from May 10, 2018

Conversation

jacobmarble
Copy link
Member

@jacobmarble jacobmarble commented May 9, 2018

Closes #9766

This change adds SHOW STATS FOR 'indexes' which returns an estimate of memory use of all indexes. Index memory use is not reported with SHOW STATS because it is a potentially expensive operation, and the implementation of SHOW STATS is also triggered every 10s to populate the _internal database.

Tested by comparing the output of this new function to the heap pprof. Results are comparable, not perfect, but pprof doesn't hold absolute truth either.

@ghost ghost assigned jacobmarble May 9, 2018
@ghost ghost added the review label May 9, 2018
@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@jacobmarble jacobmarble requested review from benbjohnson and e-dard and removed request for benbjohnson May 9, 2018 21:54
@jacobmarble
Copy link
Member Author

Working on the failing test.

@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

Copy link
Contributor

@benbjohnson benbjohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The uintptr return seems a little odd but I can't think of a better way to do it. 👍

@hercules-influx
Copy link
Contributor

During a run of megacheck the following issues were discovered:

@jacobmarble jacobmarble merged commit f1b656c into master May 10, 2018
@jacobmarble jacobmarble deleted the jm-bytes branch May 10, 2018 19:40
@ghost ghost removed the review label May 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants