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 suggest flag #85479

Merged
merged 1 commit into from
Jul 14, 2022
Merged

remove suggest flag #85479

merged 1 commit into from
Jul 14, 2022

Conversation

anmironov
Copy link
Contributor

we have a parameter suggest, it is removed since 7.0.0 #29635. But I am unsure

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

we have a parameter suggest, it is removed since 7.0.0 #29635. But I am unsure
@elasticsearchmachine
Copy link
Collaborator

@anmironov please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation.

@elasticsearchmachine elasticsearchmachine added v8.2.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Mar 30, 2022
@jtibshirani jtibshirani added >docs General docs changes :Data Management/Stats Statistics tracking and retrieval APIs labels Apr 21, 2022
@elasticmachine elasticmachine added Team:Docs Meta label for docs team Team:Data Management Meta label for data/management team labels Apr 21, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-data-management (Team:Data Management)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@lockewritesdocs lockewritesdocs removed the Team:Docs Meta label for docs team label May 10, 2022
@joegallo joegallo self-requested a review July 14, 2022 15:05
@joegallo joegallo self-assigned this Jul 14, 2022
@joegallo
Copy link
Contributor

Okay, tracing this through backwards and writing up my thought process:

  1. suggest where you've removed it is part of the index-metrics tag.
  2. And that's only used in one place, the indices stats page:
    joegallo@galactic:~/Code/elastic/elasticsearch $ git grep 'index-metric'
    docs/reference/indices/stats.asciidoc:`GET /<target>/_stats/<index-metric>`
    docs/reference/indices/stats.asciidoc:include::{es-repo-dir}/rest-api/common-parms.asciidoc[tag=index-metric]
    
  3. Which is https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-stats.html, or GET /{someindex}/_stats
  4. Testing GET /myindex/_stats/suggest does indeed result in an error, because we removed that in Remove the suggest metric from stats APIs #29635, as you correctly indicated.

So, yeah, +1, that makes sense to me.

While I was there, I noticed a couple of other things that were off, too, and I'll bring those up with the relevant stakeholders and see to fixing them. Thanks for bringing this to our attention and submitting a pull request!

@joegallo joegallo merged commit ebde65d into master Jul 14, 2022
@joegallo joegallo deleted the anmironov-patch-1 branch July 14, 2022 16:50
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Jul 15, 2022
* upstream/master: (2974 commits)
  Reserved cluster state service (elastic#88527)
  Add transport action immutable state checks (elastic#88491)
  Remove suggest flag from index stats docs (elastic#85479)
  Polling cluster formation state for master-is-stable health indicator (elastic#88397)
  Add test execution guide in yamlRestTest asciidoc (elastic#88490)
  Add troubleshooting guide for corrupt repository (elastic#88391)
  [Transform] Finetune Schedule to be less noisy on retry and retry slower (elastic#88531)
  Updatable API keys - auto-update legacy RDs (elastic#88514)
  Fix typo in TransportForceMergeAction and TransportClearIndicesCacheA… (elastic#88064)
  Fixed NullPointerException on bulk request (elastic#88358)
  Avoid needless index metadata builders during reroute (elastic#88506)
  Set metadata on request in API key noop test (elastic#88507)
  Fix passing positional args to ES in Docker (elastic#88502)
  Improve description for task api detailed param (elastic#88493)
  Support cartesian shape with doc values (elastic#88487)
  Promote usage of Subjects in Authentication class (elastic#88494)
  Add CCx 2.0 feature flag (elastic#88451)
  Reword the watcher 'always' and 'never' condition docs (elastic#86105)
  Simplify azure discovery installation docs (elastic#88404)
  Breakup FIPS CI testing jobs
  ...

# Conflicts:
#	server/src/main/java/org/elasticsearch/index/mapper/NumberFieldMapper.java
#	x-pack/plugin/mapper-aggregate-metric/src/main/java/org/elasticsearch/xpack/aggregatemetric/mapper/AggregateDoubleMetricFieldMapper.java
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 >docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team Team:Data Management Meta label for data/management team v7.17.6 v8.3.3 v8.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants