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 _all example #31711

Merged
merged 1 commit into from
Jul 2, 2018
Merged

Remove _all example #31711

merged 1 commit into from
Jul 2, 2018

Conversation

synhershko
Copy link
Contributor

Since the _all is now deprecated and removed, this example in the highlight docs is irrelevant and confusing.

Note, there are many references to the _all field in this doc, some will require a rewrite of the examples so I didn't go into doing that.

Since the `_all` is now deprecated and removed, this example in the highlight docs is irrelevant and confusing.

Note, there are many references to the `_all` field in this doc, some will require a rewrite of the examples so I didn't go into doing that.
@nik9000 nik9000 added the :Docs label Jul 2, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs

@nik9000 nik9000 self-requested a review July 2, 2018 18:05
@nik9000
Copy link
Member

nik9000 commented Jul 2, 2018

Thanks @synhershko! I agree on both counts. I'll merge and forward port the change and try and rewrite the examples.

@nik9000 nik9000 added the >docs General docs changes label Jul 2, 2018
@nik9000 nik9000 merged commit e45697c into elastic:6.3 Jul 2, 2018
nik9000 pushed a commit that referenced this pull request Jul 2, 2018
Since the `_all` is now deprecated and removed, this example in the
highlight docs is irrelevant and confusing.

Note, there are many references to the `_all` field in this doc, some
will require a rewrite of the examples so I didn't go into doing that.
@nik9000
Copy link
Member

nik9000 commented Jul 2, 2018

It looks like @debadair already rewrote the docs in master. @debadair, could you backport some of those fixes to 6.x and 6.3 for the _all field?

dnhatn added a commit that referenced this pull request Jul 4, 2018
* 6.x:
  Fix not waiting for Netty ThreadDeathWatcher in IT (#31758) (#31789)
  [Docs] Correct default window_size (#31582)
  S3 fixture should report 404 on unknown bucket (#31782)
  [ML] Limit ML filter items to 10K (#31731)
  Fixture for Minio testing (#31688)
  [ML] Return statistics about forecasts as part of the jobsstats and usage API (#31647)
  [DOCS] Add missing get mappings docs to HLRC (#31765)
  [DOCS] Starting Elasticsearch (#31701)
  Fix coerce validation_method in GeoBoundingBoxQueryBuilder (#31747)
  Painless: Complete Removal of Painless Type (#31699)
  Consolidate watcher setting update registration (#31762)
  [DOCS] Adds empty 6.3.1 release notes page
  ingest: Introduction of a bytes processor (#31733)
  [test] don't run bats tests for suse boxes (#31749)
  Add analyze API to high-level rest client (#31577)
  Implemented XContent serialisation for GetIndexResponse (#31675)
  [DOCS] Typos
  DOC: Add examples to the SQL docs (#31633)
  Add support for AWS session tokens (#30414)
  Watcher: Reenable start/stop yaml tests (#31754)
  JDBC: Fix stackoverflow on getObject and timestamp conversion (#31735)
  Support multiple system store types (#31650)
  Add write*Blob option to replace existing blob (#31729)
  Split CircuitBreaker-related tests (#31659)
  Painless: Add Context Docs (#31190)
  Docs: Remove missing reference
  Migrate scripted metric aggregation scripts to ScriptContext design (#30111)
  Watcher: Fix chain input toXcontent serialization (#31721)
  Remove _all example (#31711)
  rest-high-level: added get cluster settings (#31706)
  Docs: Match the examples in the description (#31710)
  [Docs] Correct typos (#31720)
  Extend allowed characters for grok field names (#21745) (#31653) (#31722)
  [DOCS] Check for Windows and *nix file paths (#31648)
  [ML] Validate ML filter_id (#31535)
  Fix gradle4.8 deprecation warnings (#31654)
  Update numbers to reflect 4-byte UTF-8-encoded characters (#27083)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants