bigcyy commented on issue #3220: URL: https://github.com/apache/hertzbeat/issues/3220#issuecomment-2783019705
Although this does not affect the correctness of alert results, it introduces a performance issue. For example, in a scenario with 50 Redis instances where only one is configured for alerting, the other 49 unassociated instances still get evaluated unnecessarily. I will try to fix it by updating the regex to allow optional whitespace after the comma: ``` equals\\(__labels__,\\s*\"([^\"]+)\"\\) ``` This ensures compatibility with both formats, with or without whitespace. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
