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

   ## Summary
   Consolidates the cloud inventory pagination fixes from #2684, #2610, and 
#2669 into one PR based on the current `rocketmq-studio` branch.
   
   Replaces #2684
   Replaces #2610
   Replaces #2669
   
   Closes #2678
   Closes #2609
   Closes #2667
   
   ## Changes
   - Page Tencent cloud instance and consumer group inventory by `TotalCount` 
or short-page termination instead of the old fixed 10,000-record ceiling.
   - Page Tencent ACL role and topic-subscription inventories by `TotalCount`, 
preserving short-page termination when the API omits totals.
   - Page Aliyun consumer group inventory until `totalCount` is satisfied or a 
short page is returned.
   - Add regression tests for boundary conditions at and beyond the old 
pagination ceilings.
   
   ## Verification
   - 
`JAVA_HOME=/Users/aias/Library/Java/JavaVirtualMachines/openjdk-21.0.2/Contents/Home
 mvn -q 
-Dtest=AliyunInstanceProviderTest,TencentCatalogServiceTest,TencentAclServiceTest,TencentInstanceProviderTest
 test`
   - 
`JAVA_HOME=/Users/aias/Library/Java/JavaVirtualMachines/openjdk-21.0.2/Contents/Home
 mvn -q -Dtest='org.apache.rocketmq.studio.provider.*.*Test' test`
   - 
`JAVA_HOME=/Users/aias/Library/Java/JavaVirtualMachines/openjdk-21.0.2/Contents/Home
 mvn -q -DskipTests compile`
   - `git diff --check origin/rocketmq-studio...HEAD`
   
   Full backend `mvn -q test` was also run. It currently fails on the existing 
baseline `AlertSchemaMigrationTest` issue: `Table "rmq_instance_message" not 
found` while `AlertSchemaMigration` adds `result_snapshot`; this PR does not 
touch that area.
   


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