Skip to content

[7.x] [ML] Use a new ML endpoint to estimate a model memory (#60376)#60645

Merged
darnautov merged 2 commits intoelastic:7.xfrom
darnautov:backport/7.x/pr-60376
Mar 20, 2020
Merged

[7.x] [ML] Use a new ML endpoint to estimate a model memory (#60376)#60645
darnautov merged 2 commits intoelastic:7.xfrom
darnautov:backport/7.x/pr-60376

Conversation

@darnautov
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.x:

* [ML] refactor calculate_model_memory_limit route, use estimateModelMemory endpoint

* [ML] refactor validate_model_memory_limit, migrate tests to jest

* [ML] fix typing issue

* [ML] start estimateModelMemory url with /

* [ML] fix typo, filter mlcategory

* [ML] extract getCardinalities function

* [ML] fields_service.ts

* [ML] wip getMaxBucketCardinality

* [ML] refactor and comments

* [ML] fix aggs keys with special characters, fix integration tests

* [ML] use pre-defined job types

* [ML] fallback to 0 in case max bucket cardinality receives null

* [ML] calculateModelMemoryLimit on influencers change

* [ML] fix maxModelMemoryLimit

* [ML] cap aggregation to max 1000 buckets

* [ML] rename intervalDuration
@darnautov darnautov added the backport This PR is a backport of another PR label Mar 19, 2020
@darnautov darnautov self-assigned this Mar 19, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/ml-ui (:ml)

@darnautov darnautov removed the v7.7.0 label Mar 19, 2020
@darnautov
Copy link
Copy Markdown
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@darnautov darnautov merged commit d9025ba into elastic:7.x Mar 20, 2020
@darnautov darnautov deleted the backport/7.x/pr-60376 branch March 20, 2020 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR :ml

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants