liuxuzxx commented on a change in pull request #3386:
URL: https://github.com/apache/rocketmq/pull/3386#discussion_r720611791



##########
File path: 
remoting/src/main/java/org/apache/rocketmq/remoting/protocol/RemotingSerializable.java
##########
@@ -17,11 +17,17 @@
 package org.apache.rocketmq.remoting.protocol;
 
 import com.alibaba.fastjson.JSON;
+import com.alibaba.fastjson.serializer.SerializerFeature;
+
 import java.nio.charset.Charset;
 
 public abstract class RemotingSerializable {
     private final static Charset CHARSET_UTF8 = Charset.forName("UTF-8");
 
+    static {

Review comment:
       
fastjson默认生成的key会产生不规范情况,如果只是重载方法,那样子就需要修改很多可能不规范的地方的调用,然后,改动量很大!现在只需要加这一个序列化属性,让rocketmq的所有remote调用结果全部是合法的json@我认为这是一个错误,而不算是一个优化了!




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