Skip to content

[Transform] transform can not be disabled on a subset of master-e/data nodes #52200

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

Closed
hendrikmuhs opened this issue Feb 11, 2020 · 1 comment · Fixed by #52712
Closed

[Transform] transform can not be disabled on a subset of master-e/data nodes #52200

hendrikmuhs opened this issue Feb 11, 2020 · 1 comment · Fixed by #52712
Labels
:ml/Transform Transform

Comments

@hendrikmuhs
Copy link

hendrikmuhs commented Feb 11, 2020

Affected versions:7.2 ->

Problem: Transform uses the persistent task framework and places its task on a node out of all data nodes. The placement is based on load and can not be controlled.

Mitigation: Do not disable transform on a subset of nodes. The limitation has been document in #52194

Important remark: The transform task uses very few resources, it only coordinates the data flow between search and index operations. The resource heavy tasks, be it search/data aggregations and indexing run on the data node that holds the corresponding shards.

Solution

Transform should allow to control on which nodes it runs. This can be done similar to ML which uses a node.ml setting. Transform should add a node.transform setting to explicitly disable transform on a certain node. In addition to that xpack.transform.enabled: false should implicitly disable transform assignment on that node (although we think disabling the plugin is not the right way to do it).

Related issue: #50033

@hendrikmuhs hendrikmuhs added the :ml/Transform Transform label Feb 11, 2020
@elasticmachine
Copy link
Collaborator

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

hendrikmuhs pushed a commit that referenced this issue Mar 2, 2020
…orm (#52712)

implement transform node attributes to disable transform on certain nodes and test which nodes are allowed to do remote connections

closes #52200
closes #50033
closes #48734
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this issue Mar 2, 2020
…odes and test which nodes are allowed to do remote connections

closes elastic#52200
closes elastic#50033
closes elastic#48734
hendrikmuhs pushed a commit that referenced this issue Mar 2, 2020
implement transform node attributes to disable transform on certain nodes and
test which nodes are allowed to do remote connections

closes #52200
closes #50033
closes #48734

backport #52712
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:ml/Transform Transform
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants