Skip to content

[ML] Adapt to periodic persistent task refresh#36494

Merged
droberts195 merged 4 commits intoelastic:feature-jindex-6xfrom
droberts195:adapt_to_periodic_task_refresh
Dec 14, 2018
Merged

[ML] Adapt to periodic persistent task refresh#36494
droberts195 merged 4 commits intoelastic:feature-jindex-6xfrom
droberts195:adapt_to_periodic_task_refresh

Conversation

@droberts195
Copy link
Copy Markdown

Feature branch PR

If #36069 is
merged then the approach for reallocating ML persistent tasks
after refreshing job memory requirements can be simplified.
This change begins the simplification process.

Currently WIP as #36069
is not merged.

If https://github.com/elastic/elasticsearch/pull/36069/files is
merged then the approach for reallocating ML persistent tasks
after refreshing job memory requirements can be simplified.
This change begins the simplification process.
@droberts195 droberts195 added WIP :ml Machine learning labels Dec 11, 2018
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/ml-core

Copy link
Copy Markdown
Member

@davidkyle davidkyle left a comment

Choose a reason for hiding this comment

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

LGTM

}

@TestLogging("org.elasticsearch.xpack.ml.action:TRACE,org.elasticsearch.xpack.ml.process:TRACE")
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/pull/36069")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Will this be removed in the next PR?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

No, I'll get that working before merging this PR.

@davidkyle
Copy link
Copy Markdown
Member

run gradle build tests 2

@droberts195 droberts195 merged commit e0a58c9 into elastic:feature-jindex-6x Dec 14, 2018
@droberts195 droberts195 deleted the adapt_to_periodic_task_refresh branch December 14, 2018 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:ml Machine learning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants