yyqdbngt opened a new pull request, #2224:
URL: https://github.com/apache/rocketmq-dashboard/pull/2224

   ## Summary
   
   - resolve bundled Grafana dashboards by logical UID in deterministic 
resource order
   - apply the same deterministic basename deduplication to alert-rule assets
   - keep dashboard list, lookup, and ZIP export aligned
   - keep alert asset list, lookup, and default-rule loading aligned
   - cover duplicate resources in reversed discovery order for both asset types
   
   ## Root cause
   
   Both services consumed every result from `classpath*` resource discovery. 
When multiple classpath locations supplied the same logical filename, duplicate 
dashboards or rules could be exposed, and dashboard export could attempt to 
create the same ZIP entry twice. Lookup behavior also depended on classpath 
enumeration order.
   
   The services now sort by resource description and retain one resource per 
logical basename before any operation consumes the assets.
   
   ## Validation
   
   - `mvn -f server/pom.xml 
-Dtest=GrafanaDashboardServiceTest,AlertRuleAssetServiceTest test` — 21 tests 
passed
   - Maven validate lifecycle — 0 Checkstyle violations
   
   Closes #2214


-- 
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