Simplify dubbo-common transitive dependencies (#5107)

This commit is contained in:
ken.lj 2019-09-24 16:08:00 +08:00 committed by cvictory
parent 7f95cd5290
commit 96e7c65e70
4 changed files with 7 additions and 20 deletions

View File

@ -57,25 +57,9 @@
<groupId>org.javassist</groupId>
<artifactId>javassist</artifactId>
</dependency>
<dependency>
<groupId>com.alibaba</groupId>
<artifactId>hessian-lite</artifactId>
</dependency>
<dependency>
<groupId>com.alibaba</groupId>
<artifactId>fastjson</artifactId>
</dependency>
<dependency>
<groupId>com.esotericsoftware</groupId>
<artifactId>kryo</artifactId>
</dependency>
<dependency>
<groupId>de.javakaffee</groupId>
<artifactId>kryo-serializers</artifactId>
</dependency>
<dependency>
<groupId>de.ruedigermoeller</groupId>
<artifactId>fst</artifactId>
</dependency>
</dependencies>
</project>

View File

@ -148,7 +148,7 @@ public class ApolloDynamicConfiguration implements DynamicConfiguration {
@Override
public String getProperties(String key, String group, long timeout) throws IllegalStateException {
if(StringUtils.isEmpty(group)) {
if (StringUtils.isEmpty(group)) {
return dubboConfigFile.getContent();
}
if (group.equals(url.getParameter(APPLICATION_KEY))) {

View File

@ -44,5 +44,10 @@
<artifactId>dubbo-remoting-api</artifactId>
<version>${project.parent.version}</version>
</dependency>
<dependency>
<groupId>com.alibaba</groupId>
<artifactId>hessian-lite</artifactId>
<scope>test</scope>
</dependency>
</dependencies>
</project>

View File

@ -16,8 +16,6 @@
*/
package org.apache.dubbo.common.serialize.support;
import com.esotericsoftware.kryo.Serializer;
import java.util.LinkedHashMap;
import java.util.Map;
@ -45,7 +43,7 @@ public abstract class SerializableClassRegistry {
* @param clazz object type
* @param serializer object serializer
*/
public static void registerClass(Class clazz, Serializer serializer) {
public static void registerClass(Class clazz, Object serializer) {
if (clazz == null) {
throw new IllegalArgumentException("Class registered to kryo cannot be null!");
}