tomsun28 opened a new pull request, #3314: URL: https://github.com/apache/hertzbeat/pull/3314
## What's changed? <!-- Describe Your PR Here --> - fix the calculator expr exist not work - update app-website field key, replace the `-` in the key. @zhangshenghang hi I found that the https://github.com/apache/hertzbeat/blob/master/hertzbeat-manager/src/main/resources/define/app-hugegraph.yml#L92 also has the field key name problem, i am not have the hugegraph running environment so can not update and verify. The key is a variable and must follow the variable naming rules, containing only uppercase and lowercase characters and _. Due the `-` or other like `+` are calculation symbols will be recognized in the expression. ## Checklist - [ ] I have read the [Contributing Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide) - [ ] I have written the necessary doc or comment. - [ ] I have added the necessary unit tests and all cases have passed. ## Add or update API - [ ] I have added the necessary [e2e tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have passed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@hertzbeat.apache.org For additional commands, e-mail: notifications-h...@hertzbeat.apache.org
