You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rocketmq.apache.org by GitBox <gi...@apache.org> on 2022/04/06 15:54:19 UTC

[GitHub] [rocketmq] HScarb commented on a diff in pull request #4108: [ISSUE #4107] rocketmq-tools should not depend on rocketmq-store

HScarb commented on code in PR #4108:
URL: https://github.com/apache/rocketmq/pull/4108#discussion_r844115610


##########
store/src/main/java/org/apache/rocketmq/store/stats/BrokerStatsManager.java:
##########
@@ -29,37 +30,37 @@
 
 public class BrokerStatsManager {
 
-    public static final String QUEUE_PUT_NUMS = "QUEUE_PUT_NUMS";
-    public static final String QUEUE_PUT_SIZE = "QUEUE_PUT_SIZE";
-    public static final String QUEUE_GET_NUMS = "QUEUE_GET_NUMS";
-    public static final String QUEUE_GET_SIZE = "QUEUE_GET_SIZE";
-    public static final String TOPIC_PUT_NUMS = "TOPIC_PUT_NUMS";
-    public static final String TOPIC_PUT_SIZE = "TOPIC_PUT_SIZE";
-    public static final String GROUP_GET_NUMS = "GROUP_GET_NUMS";
-    public static final String GROUP_GET_SIZE = "GROUP_GET_SIZE";
-    public static final String SNDBCK_PUT_NUMS = "SNDBCK_PUT_NUMS";
-    public static final String BROKER_PUT_NUMS = "BROKER_PUT_NUMS";
-    public static final String BROKER_GET_NUMS = "BROKER_GET_NUMS";
-    public static final String GROUP_GET_FROM_DISK_NUMS = "GROUP_GET_FROM_DISK_NUMS";
-    public static final String GROUP_GET_FROM_DISK_SIZE = "GROUP_GET_FROM_DISK_SIZE";
-    public static final String BROKER_GET_FROM_DISK_NUMS = "BROKER_GET_FROM_DISK_NUMS";
-    public static final String BROKER_GET_FROM_DISK_SIZE = "BROKER_GET_FROM_DISK_SIZE";
+    @Deprecated public static final String QUEUE_PUT_NUMS = Stats.QUEUE_PUT_NUMS;

Review Comment:
   You can move all deprecated constants to `Stats`, and remove `Stats.` in front of all constant names.
   IDEA's `Move Members` can do it quickly



-- 
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: dev-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org