Skip to content

Remove MigrationClient from High Level Rest Client#83609

Merged
joegallo merged 1 commit intoelastic:masterfrom
joegallo:remove-migrationclient
Feb 7, 2022
Merged

Remove MigrationClient from High Level Rest Client#83609
joegallo merged 1 commit intoelastic:masterfrom
joegallo:remove-migrationclient

Conversation

@joegallo
Copy link
Copy Markdown
Contributor

@joegallo joegallo commented Feb 7, 2022

Related to #83423

It was just used in the one ML test, so it was pretty easy to replace that one use with the LLRC instead.

@joegallo joegallo added >test Issues or PRs that are addressing/adding tests :Data Management/Java High Level REST Client v8.2.0 labels Feb 7, 2022
@joegallo joegallo requested a review from danhermann February 7, 2022 19:23
@elasticmachine elasticmachine added the Team:Data Management (obsolete) DO NOT USE. This team no longer exists. label Feb 7, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@joegallo joegallo mentioned this pull request Feb 7, 2022
66 tasks
@joegallo joegallo requested a review from dakrone February 7, 2022 21:53
Copy link
Copy Markdown
Member

@dakrone dakrone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@joegallo joegallo merged commit c7ec24a into elastic:master Feb 7, 2022
@joegallo joegallo deleted the remove-migrationclient branch February 7, 2022 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Team:Data Management (obsolete) DO NOT USE. This team no longer exists. >test Issues or PRs that are addressing/adding tests v8.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants