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 more fine grained memory stats from Lucene segment reader to index stats #8832

Merged
merged 1 commit into from Dec 8, 2014

Conversation

rjernst
Copy link
Member

@rjernst rjernst commented Dec 8, 2014

This is a start to exposing memory stats improvements from Lucene 5.0.
This adds the following categories of Lucene index pieces to index stats:

  • Terms
  • Stored fields
  • Term Vectors
  • Norms
  • Doc values

@rmuir
Copy link
Contributor

rmuir commented Dec 8, 2014

looks good, thank you!

@rjernst rjernst added v2.0.0-beta1 :Data Management/Stats Statistics tracking and retrieval APIs >enhancement labels Dec 8, 2014
@kimchy
Copy link
Member

kimchy commented Dec 8, 2014

LGTM, wonderful!

@@ -1225,7 +1229,13 @@ public SegmentsStats segmentsStats() {
try (final Searcher searcher = acquireSearcher("segments_stats")) {
Copy link
Contributor

Choose a reason for hiding this comment

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

can we have a unittest for this in InternalEngineTest maybe?

Copy link
Member Author

Choose a reason for hiding this comment

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

@s1monw yes I will add in a follow up commit

Copy link
Member Author

Choose a reason for hiding this comment

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

@s1monw Added a test in 0c8f5ac

This is a start to exposing memory stats improvements from Lucene 5.0.
This adds the following categories of Lucene index pieces to index stats:
* Terms
* Stored fields
* Term Vectors
* Norms
* Doc values
@rjernst rjernst merged commit fde32cc into elastic:master Dec 8, 2014
@rjernst rjernst deleted the pr/mem-stats branch January 21, 2015 23:22
@clintongormley clintongormley changed the title Stats: Add more fine grained memory stats from Lucene segment reader. Add more fine grained memory stats from Lucene segment reader to index stats Jun 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Stats Statistics tracking and retrieval APIs >enhancement v2.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants