Skip to content

Index Stats: Add support for segments stats#4102

Closed
kimchy wants to merge 1 commit intoelastic:masterfrom
kimchy:segments_stats
Closed

Index Stats: Add support for segments stats#4102
kimchy wants to merge 1 commit intoelastic:masterfrom
kimchy:segments_stats

Conversation

@kimchy
Copy link
Copy Markdown
Member

@kimchy kimchy commented Nov 5, 2013

closes #4101

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Having a handler per stats type was already the setup, but I think we can have a base class to all of them and save on this boiler plate code which all of them have.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

agreed, we should refactor it!, we need to do it in another change though...

@bleskes
Copy link
Copy Markdown
Contributor

bleskes commented Nov 6, 2013

Good addition. Left some minor comments.

@bleskes bleskes closed this Nov 6, 2013
@kimchy kimchy deleted the segments_stats branch November 10, 2013 17:58
breskeby pushed a commit to breskeby/elasticsearch that referenced this pull request Feb 11, 2026
Add test for multi-project snapshot support elastic#130000
breskeby pushed a commit to breskeby/elasticsearch that referenced this pull request Feb 11, 2026
breskeby pushed a commit to breskeby/elasticsearch that referenced this pull request Feb 11, 2026
Add test for multi-project snapshot support elastic#130000
breskeby pushed a commit to breskeby/elasticsearch that referenced this pull request Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Index Stats: Add support for segments stats

2 participants