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

   **Issue Evaluation**
   
   Category: `enhancement` | Status: **Evaluated**
   
   Thank you for this proposal. After reviewing the current codebase, the 
`MetricSample` class and the normalized metric boundary mentioned in this issue 
**do not yet exist** in `apache/rocketmq-dashboard`.
   
   This appears to be a proposal for new validation logic rather than a fix for 
existing code. The idea of rejecting non-finite (`NaN`, `±Infinity`) values at 
the metric ingestion boundary is sound defensive engineering.
   
   **Feasibility:** Feasible — would require creating a `MetricSample` model 
with finite-value validation.
   **Scope:** New code; no backward-compatibility risk.
   **Suggestion:** If this is part of a broader metrics subsystem design, 
consider bundling it with the other metric-related proposals for a single 
design discussion.
   
   ---
   *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