Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Query anomaly only for HC detector #322

Conversation

yizheliu-amazon
Copy link
Contributor

Issue #, if available:

Description of changes:
Query anomaly only for HC detector. This is for issue found during performance test: in current implementation, if there are over 10K(our upper limit) anomaly results during query time range, we only get 10K results, including both 0 and >0 result, so that we may miss anomalies with grade > 0. It is easy to get into such case for HC detector.

And actually HC detector for now only uses anomalyResults with grade > 0, thus I change to only get anomalyResults with grade > 0 for HC detector to avoid above case.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@ohltyler ohltyler left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the change

@yizheliu-amazon yizheliu-amazon merged commit 5d4f25b into opendistro-for-elasticsearch:master Oct 22, 2020
@ylwu-amzn ylwu-amzn added the enhancement Enhance current feature for better performance, user experience, etc label Oct 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Enhance current feature for better performance, user experience, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants