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

[Transform] Add a node role to node.transform #53156

Closed
hendrikmuhs opened this issue Mar 5, 2020 · 1 comment · Fixed by #53616
Closed

[Transform] Add a node role to node.transform #53156

hendrikmuhs opened this issue Mar 5, 2020 · 1 comment · Fixed by #53616
Labels
:ml/Transform Transform

Comments

@hendrikmuhs
Copy link

In #52712 we added node.transform to allow control over placing the transform task to certain nodes up to defining a dedicated transform node.

However we did not add a node role yet. In order to properly document the feature and make it visible in e.g. the cat API's, a node role should be added.

/CC @lcawl
@hendrikmuhs hendrikmuhs added the :ml/Transform Transform label Mar 5, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/ml-core (:ml/Transform)

@hendrikmuhs hendrikmuhs mentioned this issue Mar 16, 2020
Merged
hendrikmuhs pushed a commit that referenced this issue Mar 17, 2020
[Transform] add transform discovery node role (#53616)
68a698f 
Enhancement of #52712: Add a discovery node role using the letter t for transform.

Fixes #53156
hendrikmuhs pushed a commit that referenced this issue Mar 17, 2020
[Transform] add transform discovery node role (#53616)
a0314ad 
Enhancement of #52712: Add a discovery node role using the letter t for transform.

Fixes #53156
@lcawl lcawl mentioned this issue Mar 17, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:ml/Transform Transform
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Transform] add transform discovery node role hendrikmuhs/elasticsearch
2 participants
@hendrikmuhs @elasticmachine