fix bug and code style

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/remote/serialize/ProtoStuffUtils.java

@@ -17,14 +17,14 @@
 
 package org.apache.dolphinscheduler.remote.serialize;
 
+import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
+
 import io.protostuff.LinkedBuffer;
 import io.protostuff.ProtostuffIOUtil;
 import io.protostuff.Schema;
 import io.protostuff.runtime.RuntimeSchema;
 
-import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
-
 /**
  * ProtoStuffUtils
  */
@@ -49,20 +49,13 @@ public class ProtoStuffUtils {
 
     @SuppressWarnings("unchecked")
     private static <T> Schema<T> getSchema(Class<T> clazz) {
-        Schema<T> schema = (Schema<T>) schemaCache.get(clazz);
+        return (Schema<T>) schemaCache.computeIfAbsent(clazz, RuntimeSchema::createFrom);
-        if (schema == null) {
-            schema = RuntimeSchema.getSchema(clazz);
-            if (schema != null) {
-                schemaCache.put(clazz, schema);
-            }
-        }
-        return schema;
     }
 
     public static <T> T deserialize(byte[] bytes, Class<T> clazz) {
         Schema<T> schema = getSchema(clazz);
         T obj = schema.newMessage();
-        if(null==obj){
+        if (null == obj) {
             return null;
         }
         ProtostuffIOUtil.mergeFrom(bytes, obj, schema);

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/client/IRpcClient.java

@@ -24,7 +24,6 @@ import org.apache.dolphinscheduler.remote.utils.Host;
  */
 public interface IRpcClient {
 
-
     <T> T create(Class<T> clazz, Host host) throws Exception;
 
 }

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/client/RpcRequestTable.java

@@ -24,18 +24,17 @@ import java.util.concurrent.ConcurrentHashMap;
  */
 public class RpcRequestTable {
 
-
     private static ConcurrentHashMap<String, RpcRequestCache> requestMap = new ConcurrentHashMap<>();
 
-    public static void put(String requestId,RpcRequestCache rpcRequestCache){
+    public static void put(String requestId, RpcRequestCache rpcRequestCache) {
-        requestMap.put(requestId,rpcRequestCache);
+        requestMap.put(requestId, rpcRequestCache);
     }
 
-    public static RpcRequestCache get(String requestId){
+    public static RpcRequestCache get(String requestId) {
         return requestMap.get(requestId);
     }
 
-    public static void remove(String requestId){
+    public static void remove(String requestId) {
         requestMap.remove(requestId);
     }
 

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/codec/NettyDecoder.java

@@ -17,14 +17,14 @@
 
 package org.apache.dolphinscheduler.rpc.codec;
 
-import io.netty.buffer.ByteBuf;
-import io.netty.channel.ChannelHandlerContext;
-import io.netty.handler.codec.ByteToMessageDecoder;
-
 import org.apache.dolphinscheduler.remote.serialize.ProtoStuffUtils;
 
 import java.util.List;
 
+import io.netty.buffer.ByteBuf;
+import io.netty.channel.ChannelHandlerContext;
+import io.netty.handler.codec.ByteToMessageDecoder;
+
 /**
  * NettyDecoder
  */
@@ -32,7 +32,6 @@ public class NettyDecoder extends ByteToMessageDecoder {
 
     private Class<?> genericClass;
 
-
     public NettyDecoder(Class<?> genericClass) {
         this.genericClass = genericClass;
     }

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/codec/NettyEncoder.java

@@ -17,15 +17,14 @@
 
 package org.apache.dolphinscheduler.rpc.codec;
 
+import org.apache.dolphinscheduler.remote.serialize.ProtoStuffUtils;
+
 import io.netty.buffer.ByteBuf;
 import io.netty.channel.ChannelHandlerContext;
 import io.netty.handler.codec.MessageToByteEncoder;
 
-import org.apache.dolphinscheduler.remote.serialize.ProtoStuffUtils;
-
 /**
- * @author jiangli
+ * NettyEncoder
- * @date 2021-01-12 18:52
  */
 public class NettyEncoder extends MessageToByteEncoder {
 

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/config/ServiceBean.java

@@ -25,7 +25,6 @@ import java.util.List;
 import java.util.Map;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-
 import org.reflections.Reflections;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;

dolphinscheduler-remote/src/main/java/org/apache/dolphinscheduler/rpc/remote/NettyServer.java

@@ -18,10 +18,10 @@
 package org.apache.dolphinscheduler.rpc.remote;
 
 import org.apache.dolphinscheduler.remote.config.NettyServerConfig;
-import org.apache.dolphinscheduler.rpc.codec.NettyDecoder;
-import org.apache.dolphinscheduler.rpc.codec.NettyEncoder;
 import org.apache.dolphinscheduler.remote.utils.Constants;
 import org.apache.dolphinscheduler.remote.utils.NettyUtils;
+import org.apache.dolphinscheduler.rpc.codec.NettyDecoder;
+import org.apache.dolphinscheduler.rpc.codec.NettyEncoder;
 import org.apache.dolphinscheduler.rpc.common.RpcRequest;
 import org.apache.dolphinscheduler.rpc.common.RpcResponse;
 

dolphinscheduler-remote/src/test/java/org/apache/dolphinscheduler/rpc/UserService.java

@@ -23,7 +23,7 @@ import org.apache.dolphinscheduler.rpc.base.RpcService;
  * UserService
  */
 @RpcService("IUserService")
-public class UserService  implements IUserService{
+public class UserService implements IUserService {
 
     @Override
     public Boolean say(String s) {