[Dubbo-apache#2162]Correct the reference retries default value 0 to 2 (…

…apache#2183) * Correct the reference retries default value 0 to 2 * fix the unit test error
ChangFeng2015 · Aug 15, 2018 · f2188d3 · f2188d3
1 parent f364a44
commit f2188d3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/dubbo-compatible/src/main/java/com/alibaba/dubbo/config/annotation/Reference.java b/dubbo-compatible/src/main/java/com/alibaba/dubbo/config/annotation/Reference.java
@@ -75,7 +75,7 @@
 
     String layer() default "";
 
-    int retries() default 0;
+    int retries() default 2;
 
     String loadbalance() default "";
 

diff --git a/...o-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/annotation/Reference.java b/...o-config/dubbo-config-api/src/main/java/org/apache/dubbo/config/annotation/Reference.java
@@ -164,7 +164,7 @@
      *
      * @see Constants#DEFAULT_RETRIES
      */
-    int retries() default 0;
+    int retries() default 2;
 
     /**
      * Load balance strategy, legal values include: random, roundrobin, leastactive

diff --git a/...che/dubbo/config/spring/beans/factory/annotation/AnnotationPropertyValuesAdapterTest.java b/...che/dubbo/config/spring/beans/factory/annotation/AnnotationPropertyValuesAdapterTest.java
@@ -113,7 +113,7 @@ public Map<String, String> convert(String[] source) {
         Assert.assertEquals("ondisconnect", referenceBean.getOndisconnect());
         Assert.assertEquals("owner", referenceBean.getOwner());
         Assert.assertEquals("layer", referenceBean.getLayer());
-        Assert.assertEquals(Integer.valueOf(2), referenceBean.getRetries());
+        Assert.assertEquals(Integer.valueOf(1), referenceBean.getRetries());
         Assert.assertEquals("random", referenceBean.getLoadbalance());
         Assert.assertEquals(true, referenceBean.isAsync());
         Assert.assertEquals(Integer.valueOf(1), referenceBean.getActives());
@@ -147,7 +147,7 @@ private static class TestBean {
                 check = false, init = true, lazy = true, stubevent = true,
                 reconnect = "reconnect", sticky = true, proxy = "javassist", stub = "stub",
                 cluster = "failover", connections = 1, callbacks = 1, onconnect = "onconnect",
-                ondisconnect = "ondisconnect", owner = "owner", layer = "layer", retries = 2,
+                ondisconnect = "ondisconnect", owner = "owner", layer = "layer", retries = 1,
                 loadbalance = "random", async = true, actives = 1, sent = true,
                 mock = "mock", validation = "validation", timeout = 2, cache = "cache",
                 filter = {"default", "default"}, listener = {"default", "default"}, parameters = {"key1", "value1"}, application = "application",