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.

Sign up for GitHub

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

Jump to bottom

Implement cat.transform #4526

Closed
wants to merge 7 commits into from
Closed

Implement cat.transform #4526

wants to merge 7 commits into from

Conversation

codebrain
Copy link
Contributor

codebrain and others added 6 commits March 27, 2020 16:22
@codebrain
Update specifications for 7.7.0
d38df95
@codebrain
Generate all beta and experimental APIs in the LLC
510eb97
@github-actions @codebrain
Fix API generator (#4456) (#4514) (#4517)
0a3d8fd 
This commit fixes the API generator:

1. The collection of high level endpoints were being used in some
of the low level generators
2. High level endpoints excluded need to be compared to the list by filename

Co-authored-by: Stuart Cam <stuart.cam@elastic.co>

(cherry-picked from commit 04ed38b)

Co-authored-by: Russ Cam <russ.cam@elastic.co>
(cherry picked from commit 145869b)
@codebrain
Regenerate APIs using fixed generator
8ed85df
@codebrain
Update API list to categorise implementations for 7.7
59d38d3
@codebrain
Implement cat.transform
90ce6c8
@codebrain codebrain mentioned this pull request Mar 30, 2020
Closed
38 tasks
Mpdreamz
Mpdreamz approved these changes Mar 30, 2020
View reviewed changes
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

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

LGTM, but we should wait for: elastic/elasticsearch#54438

@codebrain codebrain changed the base branch from feature/7.7/update-7.7-specs to 7.7 March 30, 2020 23:37
@cla-checker-service
Copy link

❌ Author of the following commits did not sign a Contributor Agreement:
0a3d8fd

Please, read and sign the above mentioned agreement if you want to contribute to this project

@codebrain
Copy link
Contributor Author

CatTransformRecord needs updating: https://www.elastic.co/guide/en/elasticsearch/reference/7.7/cat-transforms.html

@codebrain codebrain mentioned this pull request Apr 3, 2020
Merged
@codebrain
Copy link
Contributor Author

Closed in favor of #4557

@codebrain codebrain closed this Apr 6, 2020
@russcam russcam deleted the feature/7.7/apis/cat-transform branch May 27, 2020 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Mpdreamz Mpdreamz Mpdreamz requested changes

Assignees
No one assigned
Labels
backport main
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@codebrain @Mpdreamz