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
[IOTDB-1319] Trigger module: alert manager sink #3057
Conversation
ac188e6
to
3d0d451
Compare
Please rename this PR to: [IOTDB-1319] Trigger module: alert manager sink |
example/trigger/src/main/java/org/apache/iotdb/trigger/AlertingTriggerExample.java
Outdated
Show resolved
Hide resolved
example/trigger/src/main/java/org/apache/iotdb/trigger/AlertingTriggerExample.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerConfiguration.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerEvent.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerEvent.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/ts/TimeSeriesConfiguration.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/ts/TimeSeriesEvent.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/ts/TimeSeriesHandler.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerHandler.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerHandler.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/apache/iotdb/db/sink/alertmanager/AlertManagerConfiguration.java
Outdated
Show resolved
Hide resolved
2b2d85e
to
db2141a
Compare
annotations.put("summary", "high temperature"); | ||
annotations.put("description", "{{.alertname}}: {{.series}} is {{.value}}"); | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we override onStart() / onStop()
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can the client be reopened?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks for your contribution.
Description
Content1 ...
Content2 ...
Content3 ...
This PR has:
for an unfamiliar reader.
for code coverage.
Key changed/added classes (or packages if there are too many classes) in this PR