Skip to content

Conversation

@hendrikmuhs
Copy link

remove the indexer threadpool and use the generic threadpool instead(The indexer threadpool was only used on start)

fixes #81796

… the

indexer threadpool was only used on start.

fixes elastic#81796
@elasticmachine elasticmachine added the Team:ML Meta label for the ML team label Dec 20, 2021
@elasticmachine
Copy link
Collaborator

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

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

@Override
public boolean maybeTriggerAsyncJob(long now) {
boolean triggered;
// threadpool: trigger_engine_scheduler if triggered from the scheduler
Copy link
Contributor

Choose a reason for hiding this comment

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

What is the alternative (if not triggered from the scheduler)?

Copy link
Author

Choose a reason for hiding this comment

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

this is also called by the transform task when it starts, e.g. as a result of _start or if the transform re-initializes due to a task re-location (failover or upgrade).

@hendrikmuhs
Copy link
Author

@elasticmachine update branch

@hendrikmuhs
Copy link
Author

@elasticmachine update branch

@hendrikmuhs hendrikmuhs merged commit c9113a5 into elastic:master Dec 21, 2021
@hendrikmuhs hendrikmuhs deleted the transform-remove-indexer-threadpool branch December 21, 2021 11:42
hendrikmuhs pushed a commit to hendrikmuhs/elasticsearch that referenced this pull request Feb 4, 2022
…on (elastic#81912)

remove the indexer threadpool and use the generic threadpool instead(The indexer threadpool was only used on start)

fixes elastic#81796
hendrikmuhs pushed a commit that referenced this pull request Feb 7, 2022
…mitation (#83539)

remove the indexer threadpool and use the generic threadpool instead(The indexer threadpool was only used on start)

fixes #81796
backport #81912
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :ml/Transform Transform Team:ML Meta label for the ML team v7.17.1 v8.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Transform] Transforms can get stopped during a rolling upgrade for many(>8) running transforms

4 participants