This is an automated email from the ASF dual-hosted git repository.

lizhimins pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/rocketmq.git


The following commit(s) were added to refs/heads/develop by this push:
     new 8242c1e9d2 [ISSUE #10562] Optimize 
PopConsumerServiceTest#transferToFsStoreTest to avoid unnecessary background 
threads (#10563)
8242c1e9d2 is described below

commit 8242c1e9d223c33b68fe1c66d129e83e86f5eba0
Author: Quan <[email protected]>
AuthorDate: Wed Jul 1 14:03:54 2026 +0800

    [ISSUE #10562] Optimize PopConsumerServiceTest#transferToFsStoreTest to 
avoid unnecessary background threads (#10563)
    
    - Replace consumerService.start() with 
consumerService.getPopConsumerStore().start()
    - Replace consumerService.shutdown() with 
consumerService.getPopConsumerStore().shutdown()
    - Avoid starting revive/cache background threads that are not needed for 
transferToFsStore verification
    - Reduces test execution time from ~60s to ~1.3s
---
 .../java/org/apache/rocketmq/broker/pop/PopConsumerServiceTest.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/broker/src/test/java/org/apache/rocketmq/broker/pop/PopConsumerServiceTest.java
 
b/broker/src/test/java/org/apache/rocketmq/broker/pop/PopConsumerServiceTest.java
index 089d2c1f22..44189744b4 100644
--- 
a/broker/src/test/java/org/apache/rocketmq/broker/pop/PopConsumerServiceTest.java
+++ 
b/broker/src/test/java/org/apache/rocketmq/broker/pop/PopConsumerServiceTest.java
@@ -464,10 +464,10 @@ public class PopConsumerServiceTest {
             .thenReturn(CompletableFuture.completedFuture(
                 new PutMessageResult(PutMessageStatus.PUT_OK, new 
AppendMessageResult(AppendMessageStatus.PUT_OK))));
 
-        consumerService.start();
+        consumerService.getPopConsumerStore().start();
         consumerService.getPopConsumerStore().writeRecords(consumerRecordList);
         consumerService.transferToFsStore();
-        consumerService.shutdown();
+        consumerService.getPopConsumerStore().shutdown();
     }
 
     @Test

Reply via email to