Defer the shutdown behavior of popConsumerService to prevent JVM crash (#9889)

Change-Id: Iaa23996d94f81ed3250b8d528338e8502a35cada

Co-authored-by: RongtongJin <[email protected]>

Author: RongtongJin

broker/src/main/java/org/apache/rocketmq/broker/BrokerController.java

@@ -1442,10 +1442,6 @@ protected void shutdownBasicService() {
             this.pullRequestHoldService.shutdown();
         }
 
-        if (this.popConsumerService != null) {
-            this.popConsumerService.shutdown();
-        }
-
         if (this.popMessageProcessor.getPopLongPollingService() != null) {
             this.popMessageProcessor.getPopLongPollingService().shutdown();
         }
@@ -1641,6 +1637,10 @@ protected void shutdownBasicService() {
             }
         }
 
+        if (this.popConsumerService != null) {
+            this.popConsumerService.shutdown();
+        }
+
         if (this.messageStore != null) {
             this.messageStore.shutdown();
         }