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.

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

[7.2][ML] Improve hard_limit audit message (#486) #487

Merged
merged 1 commit into from
May 18, 2019

Conversation

edsavage
Copy link
Contributor

relates elastic/elasticsearch#42086

Backports #486

Add the current model memory limit and the number of bytes in
excess of that at the point of the last allocation failure to the model
size stats. These will be used to construct a (hopefully) more
informative hard_limit audit message.

The reported memory usage is also scaled to take into account the byte
limit margin, which is in play in the initial period of a jobs' lifetime
and is used to scale down the high memory limit. This should give a more
accurate representation of how close the memory usage is to the high
limit.

relates elastic/elasticsearch#42086

closes elastic/elasticsearch#38034
@edsavage edsavage merged commit 7a782a9 into elastic:7.x May 18, 2019
droberts195 added a commit that referenced this pull request May 19, 2019
droberts195 added a commit that referenced this pull request May 20, 2019
@edsavage edsavage deleted the improve_hard_limit_audit_message_7x branch May 22, 2019 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant