[GitHub] ymwneu commented on a change in pull request #537: [ISSUE #525] Support the message track

[GitBox <gi...@apache.org>]

ymwneu commented on a change in pull request #537: [ISSUE #525] Support the message track
URL: https://github.com/apache/rocketmq/pull/537#discussion_r234510377
 
 

 ##########
 File path: client/src/main/java/org/apache/rocketmq/client/producer/DefaultMQProducer.java
 ##########
 @@ -119,40 +129,65 @@
      */
     private int maxMessageSize = 1024 * 1024 * 4; // 4M
 
+    /**
+     * Interface of asynchronous transfer data
+     */
+    private AsyncDispatcher traceDispatcher = null;
+
     /**
      * Default constructor.
      */
     public DefaultMQProducer() {
-        this(MixAll.DEFAULT_PRODUCER_GROUP, null);
+        this(MixAll.DEFAULT_PRODUCER_GROUP, null,false);
     }
 
     /**
      * Constructor specifying both producer group and RPC hook.
      *
      * @param producerGroup Producer group, see the name-sake field.
      * @param rpcHook RPC hook to execute per each remoting command execution.
+     * @param msgTraceSwitch switch flag instance for message track trace
      */
-    public DefaultMQProducer(final String producerGroup, RPCHook rpcHook) {
+    public DefaultMQProducer(final String producerGroup, RPCHook rpcHook, boolean msgTraceSwitch) {
 
 Review comment:
   Add a new method instead of modify the old one.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services