RockteMQ-AI commented on issue #1957:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/1957#issuecomment-5263006835

   **Issue Evaluation**
   
   Category: `type/bug` | Status: **Needs Info**
   
   The issue describes a clear contract mismatch between the frontend form and 
backend validation for alert rules. However, the referenced files do not exist 
on the `master` branch:
   
   - `web/src/pages/ops/alerts.tsx` — no `web/` directory exists; frontend is 
in `frontend-new/`
   - 
`server/src/main/java/org/apache/rocketmq/studio/ops/alert/AlertRuleRequestDTO.java`
 — Java sources are under `src/main/`, not `server/src/main/`
   - 
`server/src/main/java/org/apache/rocketmq/studio/ops/alert/AlertService.java` — 
same path discrepancy
   
   The only alert-related code found on `master` is 
`src/main/java/org/apache/rocketmq/dashboard/task/MonitorTask.java`, which does 
not contain the DTO or service described.
   
   **To proceed with verification, please clarify:**
   - [ ] Which branch or PR contains the Track 1 alert-rule implementation?
   - [ ] Are the file paths relative to a different repository or fork?
   
   If this is a feature branch under development, the fix should be applied 
there before merge. The described defect (localized display text used as API 
values) is a valid concern and the expected behavior (stable identifiers like 
`rocketmq_disk_use_ratio` and `5m`) is correct.
   
   ---
   *Automated evaluation by RockteMQ-AI*


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to