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

REST high-level client: add delete by query API #32782

Merged
merged 4 commits into from
Sep 4, 2018

Conversation

sohaibiftikhar
Copy link
Contributor

@sohaibiftikhar sohaibiftikhar commented Aug 10, 2018

Relates to #27205

@javanna
Copy link
Member

javanna commented Aug 10, 2018

add to whitelist

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@prasadcomp
Copy link

Do you know what Elasticsearch version these APIs will be targeted? I am in process of migrating our code from transport APIs to High level rest APIs as we want to avoid the tight coupling between ES transport client version and ES server version.

I am very much interested in DeleteByQuery API and would like to know the timeline these APIs will be available?

Any plans of porting over DeleteByQueryRequestBuilder to REST APIs?

@nik9000
Copy link
Member

nik9000 commented Sep 1, 2018

We won't port the Builder classes to the REST API though you can still use them if you'd like by calling request on them. Building them is a little harder because they are tied to the transport client so we suggest not using them.

I'm hoping to get this merged to the 6.x branch in time for the 6.5 release.

@nik9000 nik9000 merged commit 761e8c4 into elastic:master Sep 4, 2018
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Sep 4, 2018
…e-default-distribution

* elastic/master: (213 commits)
  ML: Fix build after HLRC change
  Fix inner hits retrieval when stored fields are disabled (_none_) (elastic#33018)
  SQL: Show/desc commands now support table ids (elastic#33363)
  Mute testValidateFollowingIndexSettings
  HLRC: Add delete by query API (elastic#32782)
  [ML] The sort field on get records should default to the record_score (elastic#33358)
  [ML] Minor improvements to categorization Grok pattern creation (elastic#33353)
  [DOCS] fix a couple of typos (elastic#33356)
  Disable assemble task instead of removing it (elastic#33348)
  Simplify the return type of FieldMapper#parse. (elastic#32654)
  [ML] Delete forecast API (elastic#31134) (elastic#33218)
  Introduce private settings (elastic#33327)
  [Docs] Add search timeout caveats (elastic#33354)
  TESTS: Fix Race Condition in Temp Path Creation (elastic#33352)
  Fix from_range in search_after in changes snapshot (elastic#33335)
  TESTS+DISTR.: Fix testIndexCheckOnStartup Flake (elastic#33349)
  Null completion field should not throw IAE (elastic#33268)
  Adds code to help with IndicesRequestCacheIT failures (elastic#33313)
  Prevent NPE parsing the stop datafeed request. (elastic#33347)
  HLRC: Add ML get overall buckets API (elastic#33297)
  ...
nik9000 pushed a commit that referenced this pull request Sep 4, 2018
Adds the delete-by-query API to the High Level REST Client.
@nik9000
Copy link
Member

nik9000 commented Sep 4, 2018

OK @sohaibiftikhar! That is the last of them! Thanks again!

@sohaibiftikhar sohaibiftikhar deleted the delete_by_query branch September 5, 2018 08:33
@jimczi jimczi 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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants