Skip to content

[7.x][ML] Limit categorization memory usage#1176

Merged
droberts195 merged 1 commit intoelastic:7.xfrom
droberts195:categorization_memory_limiting_7x
Apr 28, 2020
Merged

[7.x][ML] Limit categorization memory usage#1176
droberts195 merged 1 commit intoelastic:7.xfrom
droberts195:categorization_memory_limiting_7x

Conversation

@droberts195
Copy link
Copy Markdown

Anomaly detection jobs have a model_memory_limit setting. This is
supposed to restrict the amount of memory the job can use, however,
in the past the limit only applied to anomaly detection and not to
categorization.

This change applies memory limiting to categorization as follows:

  • When a job is in hard_limit status no new categories will be
    created. The input document that could not be categorized is
    discarded as it cannot take part in anomaly detection without a
    category. The failed_category_count statistic is incremented
    each time this happens.
  • When a job is in soft_limit status, we stop recording examples
    for the category.

Backport of #1167

Anomaly detection jobs have a model_memory_limit setting.  This is
supposed to restrict the amount of memory the job can use, however,
in the past the limit only applied to anomaly detection and not to
categorization.

This change applies memory limiting to categorization as follows:

- When a job is in hard_limit status no new categories will be
  created.  The input document that could not be categorized is
  discarded as it cannot take part in anomaly detection without a
  category.  The failed_category_count statistic is incremented
  each time this happens.
- When a job is in soft_limit status, we stop recording examples
  for the category.

Backport of elastic#1167
@droberts195 droberts195 merged commit 6abc076 into elastic:7.x Apr 28, 2020
@droberts195 droberts195 deleted the categorization_memory_limiting_7x branch April 28, 2020 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant