Skip to content
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

feat(rule-engine): 告警场景2.2相关功能更新 #552

Merged
merged 5 commits into from
Aug 21, 2024

Conversation

liu4410
Copy link
Contributor

@liu4410 liu4410 commented Aug 12, 2024

No description provided.

@zhou-hao zhou-hao requested a review from bestfeng1020 August 12, 2024 10:52
@bestfeng1020
Copy link
Contributor

社区版不支持消息预处理器

@zhou-hao
Copy link
Member

似乎没有实质性的更新内容吧?

@liu4410
Copy link
Contributor Author

liu4410 commented Aug 13, 2024

似乎没有实质性的更新内容吧?

新增了告警持续时间相关字段,告警可以关联场景联动某个具体的条件分支,

@zhou-hao
Copy link
Member

似乎没有实质性的更新内容吧?

新增了告警持续时间相关字段,告警可以关联场景联动某个具体的条件分支,

但是似乎还有很多与这些功能无关的类?

@zhou-hao
Copy link
Member

@bestfeng1020

@bestfeng1020
Copy link
Contributor

@zhou-hao

@zhou-hao
Copy link
Member

社区版测试过了么

@bestfeng1020
Copy link
Contributor

冒烟测试过了,全面测试等测试组空了后做

@bestfeng1020
Copy link
Contributor

@zhou-hao


import javax.annotation.Nonnull;

public class ShakeLimitFlux<T> extends FluxOperator<T, ShakeLimitResult<T>> {
Copy link
Member

Choose a reason for hiding this comment

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

没有使用的类

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已删除

* 采集器服务
* @see CrudCommandSupport
*/
String collectorService = "collectorService";
Copy link
Member

Choose a reason for hiding this comment

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

社区版没有此功能

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个类已删除

* 插件服务
* @see CrudCommandSupport
*/
String pluginService = "pluginService";
Copy link
Member

Choose a reason for hiding this comment

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

社区版没有此功能

pom.xml Outdated
@@ -707,12 +714,12 @@
<repository>
<id>releases</id>
<name>Nexus Release Repository</name>
<url>https://nexus.jetlinks.cn/content/repositories/releases/</url>
<url>https://nexus.jetlinks.cn/content/repositories/releases-pro/</url>
Copy link
Member

Choose a reason for hiding this comment

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

地址错误

pom.xml Outdated
@@ -32,6 +32,8 @@
<reactor.ql.version>1.0.16</reactor.ql.version>
<!-- https://github.com/jetlinks/jetlinks-plugin -->
<jetlinks.plugin.version>1.0.1</jetlinks.plugin.version>
<!-- https://github.com/jetlinks/jetlinks-sdk -->
<jetlinks.sdk.version>1.0.0-SNAPSHOT</jetlinks.sdk.version>
Copy link
Member

Choose a reason for hiding this comment

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

1.0.0

@zhou-hao zhou-hao merged commit f754740 into jetlinks:master Aug 21, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants