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

Remove the suggest metric from stats APIs #29635

Merged
merged 3 commits into from Apr 24, 2018

Conversation

Projects
None yet
4 participants
@jasontedor
Copy link
Member

commented Apr 20, 2018

This metric previously existed for backwards compatibility reasons although the suggest stats were folded into search stats. This metric was deprecated in 6.3.0 and this commit removes them for 7.0.0.

Closes #29589

Remove the suggest metric from stats APIs
This metric previously existed for backwards compatibility reasons
although the suggest stats were folded into search stats. This metric
was deprecated in 6.3.0 and this commit removes them for 7.0.0.
@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Apr 20, 2018

@jasontedor jasontedor merged commit 2c3e71f into elastic:master Apr 24, 2018

3 checks passed

CLA Commit author is a member of Elasticsearch
Details
elasticsearch-ci Build finished.
Details
elasticsearch-ci/packaging-sample Build finished.
Details

@jasontedor jasontedor deleted the jasontedor:remove-suggest-metric branch Apr 24, 2018

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Apr 25, 2018

Merge branch 'master' into set-java-home-setup-commands
* master:
  [Test] Fix docs check for DEB package in packaging tests (elastic#30126)
  Painless: Docs Clean Up (elastic#29592)
  Fixes Eclipse build for sql jdbc project (elastic#30114)
  Remove reference to `not_analyzed`.
  [Docs] Add community analysis plugin (elastic#29612)
  Remove the suggest metric from stats APIs (elastic#29635)
  Fix byte size value equals/hash code test (elastic#29643)
  Upgrade to Gradle 4.7 (elastic#29644)

jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Apr 25, 2018

Merge branch 'master' into remove-animal-sniffer
* master:
  [Test] Fix docs check for DEB package in packaging tests (elastic#30126)
  Painless: Docs Clean Up (elastic#29592)
  Fixes Eclipse build for sql jdbc project (elastic#30114)
  Remove reference to `not_analyzed`.
  [Docs] Add community analysis plugin (elastic#29612)
  Remove the suggest metric from stats APIs (elastic#29635)
  Fix byte size value equals/hash code test (elastic#29643)
  Upgrade to Gradle 4.7 (elastic#29644)

@colings86 colings86 added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.