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

[Transform] Improve transform upgrader #79601

Merged
merged 13 commits into from
Oct 25, 2021

Conversation

hendrikmuhs
Copy link

@hendrikmuhs hendrikmuhs commented Oct 21, 2021

This changes fixes some issues with the upgrader. It inlines the index name resolver as wildcard deletes are not supported if action.destructive_requires_name is set to true(default). It changes response to always return all counters in order to avoid an empty response. After documentation has been created, it switches the deprecation URL to the upgrade API docs.

Notes

  • the REFERENCE_BASE_URL must be changed on the backport
  • non-issue as the upgrade has not been released yet, however this is technically bugfixing some upgrader issues

@hendrikmuhs hendrikmuhs force-pushed the transform-upgrade-enhancements branch from 4d23596 to 17c6b4e Compare October 21, 2021 13:25
@hendrikmuhs hendrikmuhs marked this pull request as ready for review October 21, 2021 14:43
@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Oct 21, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (Team:ML)

@hendrikmuhs hendrikmuhs requested a review from lcawl October 21, 2021 14:43
Copy link
Contributor

@lcawl lcawl left a comment

Choose a reason for hiding this comment

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

Minor typo, otherwise text LGTM

lcawl added a commit to lcawl/elasticsearch that referenced this pull request Oct 21, 2021
@lcawl
Copy link
Contributor

lcawl commented Oct 21, 2021

the REFERENCE_BASE_URL must be changed on the backport

If you implement my suggestion (
#79601 (comment)), es-8-upgrade-transforms should be changed to es-7-upgrade-transforms in the backport.

Hendrik Muhs and others added 2 commits October 25, 2021 08:18
…transform/TransformDeprecations.java

Co-authored-by: Lisa Cawley <[email protected]>
@hendrikmuhs
Copy link
Author

@elasticmachine update branch

Hendrik Muhs added 2 commits October 25, 2021 09:53
Copy link
Contributor

@przemekwitek przemekwitek left a comment

Choose a reason for hiding this comment

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

LGTM

@hendrikmuhs hendrikmuhs merged commit fd3eede into elastic:master Oct 25, 2021
@hendrikmuhs hendrikmuhs deleted the transform-upgrade-enhancements branch October 25, 2021 09:01
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this pull request Oct 25, 2021
This changes fixes some issues with the upgrader. It inlines the index name resolver as wildcard
deletes are not supported if action.destructive_requires_name is set to true(default). It changes
response to always return all counters in order to avoid an empty response. After documentation has
been created, it switches the deprecation URL to the upgrade API docs.
elasticsearchmachine pushed a commit that referenced this pull request Oct 25, 2021
* [Transform] Improve transform upgrader (#79601)

This changes fixes some issues with the upgrader. It inlines the index name resolver as wildcard
deletes are not supported if action.destructive_requires_name is set to true(default). It changes
response to always return all counters in order to avoid an empty response. After documentation has
been created, it switches the deprecation URL to the upgrade API docs.

* change url to 7.16 specific one

* fix 7.x specific test

* fix put mapping call
lockewritesdocs pushed a commit to lockewritesdocs/elasticsearch that referenced this pull request Oct 28, 2021
This changes fixes some issues with the upgrader. It inlines the index name resolver as wildcard
deletes are not supported if action.destructive_requires_name is set to true(default). It changes
response to always return all counters in order to avoid an empty response. After documentation has
been created, it switches the deprecation URL to the upgrade API docs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants