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

add throttle on action level #45

Merged
merged 11 commits into from
May 14, 2019

Conversation

ylwu-amzn
Copy link
Contributor

Issue opendistro-for-elasticsearch/alerting#14:

Add throttle on action level.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@ylwu-amzn ylwu-amzn requested review from dbbaughe and mihirsoni May 10, 2019 00:42
mihirsoni
mihirsoni previously approved these changes May 10, 2019
Copy link
Contributor

@mihirsoni mihirsoni 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 changes.

mihirsoni
mihirsoni previously approved these changes May 14, 2019
Copy link
Contributor

@mihirsoni mihirsoni 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 changes.

public/utils/validate.js Outdated Show resolved Hide resolved
Copy link
Contributor

@dbbaughe dbbaughe 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 Yaliang!

@ylwu-amzn ylwu-amzn merged commit 1780970 into opendistro-for-elasticsearch:master May 14, 2019
@ylwu-amzn ylwu-amzn deleted the throttle branch May 14, 2019 21:55
@leoskyrocker
Copy link

Hey guys this is amazin' Just curious, how often do you release? Can we make a release for this soon, and once released, what does it take to get it into AWS elasticsearch??

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