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

Conversation

yizheliu-amazon
Copy link
Contributor

Issue #, if available:

Description of changes:

  • Improve performance of fetching anomalies when there are too many anomaly results
  • Fix detector stopped status message

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

@ohltyler
Copy link
Contributor

ohltyler commented May 8, 2020

Is there any broken UT we can easily fix?

@yizheliu-amazon
Copy link
Contributor Author

Is there any broken UT we can easily fix?

yeah. working on the fix now. will send out new commit.

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.

Great, LGTM!

Copy link
Contributor

@ylwu-amzn ylwu-amzn left a comment

Choose a reason for hiding this comment

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

LGTM

@yizheliu-amazon yizheliu-amazon merged commit b1f6596 into opendistro-for-elasticsearch:master May 8, 2020
yizheliu-amazon added a commit that referenced this pull request May 8, 2020
* Improve performance of fetching anomalies when there are too many anomaly results

* Fix detector stopped status message

* Fix unit test and pagination on Dashboard
yizheliu-amazon added a commit that referenced this pull request May 8, 2020
* Improve performance of fetching anomalies when there are too many anomaly results

* Fix detector stopped status message

* Fix unit test and pagination on Dashboard
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants