-
Notifications
You must be signed in to change notification settings - Fork 997
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
[Feature] Solve alarm current limiting #2142
Comments
+1 good suggestion. |
I think a core point is that we should provide the user with some rules for alerting, not write dead by default |
Thank you for your suggestion. I think adding alarm rules is a good proposal. I also heard some requirements for alarm rules, and I will think about it again and improve this idea. |
I'll add a little feature: |
Thank you for your proposal. I'll take it into consideration. |
Search before asking
Description
In the actual production environment, streampark may have current restriction when using software alarms such as
flash book and nail, which may result in some alarm messages not being sent.
In my opinion, alarms should not be sent because they are an important indicator of task status.I want to solve the alarm problem by adding a blocking queue. When the task fails, the alarm is not sent at the first time, but added to the blocking queue,Then send the alarm through a separate alarm thread. Of course, this change may affect the effectiveness of the alarm. After my test, this change can solve the problem of alarm current limiting.
Usage Scenario
Task alarm
Related issues
No response
Are you willing to submit a PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: