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

   ### Summary
   
   `AliyunConverters.toQueueProgressRows` read `topicLagMap` but ignored 
`liteTopicLagMap`. A LiteTopic-only group therefore lost its per-topic 
breakdown. When `totalLag.readyCount` was present, callers saw only an 
unattributed aggregate row; when it was absent, they saw no row.
   
   This change converts both maps through the existing topic-row builder. The 
aggregate fallback still applies only when neither map supplies a row, so it 
cannot double count the topic breakdown. The returned row order follows the SDK 
maps' iteration order and is not a contract of this converter.
   
   ### Validation
   
   The original implementation was verified with `mvn -B -ntp test 
-Dtest=AliyunConvertersLagTest`: six tests passed, including the new 
LiteTopic-only and mixed-map cases. This review update changes only a source 
comment and the commit description. The configured remote test server timed out 
during this update; current CI is the remaining validation.
   
   Related: #5108 (closed as not planned).
   


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