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

[DOCS] Removed like-text and ids params from MLT. Closes #28128 #31370

Merged
merged 1 commit into from
Jun 19, 2018

Conversation

Sue-Gallagher
Copy link
Contributor

@Sue-Gallagher Sue-Gallagher commented Jun 15, 2018

Removes like-text and ids from MLT per Breaking Changes 6.0

@pcsanwald
Copy link
Contributor

LGTM. @Sue-Gallagher I think you will want to label this appropriately >docs and also 7.0.0, and, once merged, backport it to 6.4.0. If you haven't done backporting before, myself or @debadair or anyone from the ES team would be happy to help.

@Sue-Gallagher Sue-Gallagher added >docs General docs changes and removed >docs General docs changes v7.0.0 labels Jun 15, 2018
@Sue-Gallagher Sue-Gallagher deleted the mlt-fix2 branch June 15, 2018 23:44
@Sue-Gallagher Sue-Gallagher restored the mlt-fix2 branch June 19, 2018 20:47
@Sue-Gallagher Sue-Gallagher reopened this Jun 19, 2018
@Sue-Gallagher Sue-Gallagher merged commit b44e1c1 into elastic:master Jun 19, 2018
dnhatn added a commit that referenced this pull request Jun 20, 2018
* master:
  [DOCS] Omit shard failures assertion for incompatible responses  (#31430)
  [DOCS] Move licensing APIs to docs (#31445)
  Add Delete Snapshot High Level REST API
  Remove QueryCachingPolicy#ALWAYS_CACHE (#31451)
  [Docs] Extend Homebrew installation instructions (#28902)
  Choose JVM options ergonomically
  [Docs] Mention ip_range datatypes on ip type page (#31416)
  Multiplexing token filter (#31208)
  Fix use of time zone in date_histogram rewrite (#31407)
  Core: Remove index name resolver from base TransportAction (#31002)
  [DOCS] Fixes code snippet testing for machine learning (#31189)
  [DOCS] Removed  and  params from MLT. Closes #28128 (#31370)
  Security: fix joining cluster with production license (#31341)
  Unify http channels and exception handling (#31379)
  [DOCS] Moves the info API to docs (#31121)
  Preserve response headers on cluster update task (#31421)
  [DOCS] Add code snippet testing for more ML APIs (#31404)
  Do not preallocate bytes for channel buffer (#31400)
  Docs: Advice for reindexing many indices (#31279)
  Mute HttpExporterTests#testHttpExporterShutdown test Tracked by #31433
  Docs: Add note about removing prepareExecute from the java client (#31401)
  Make release notes ignore the `>test-failure` label. (#31309)
@milani
Copy link

milani commented Nov 29, 2018

Is this backported? The documents at https://www.elastic.co/guide/en/elasticsearch/reference/6.x/query-dsl-mlt-query.html still show docs and ids as parameters.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants