hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@apache.org
Subject hbase git commit: HBASE-16238 It's useless to catch SESSIONEXPIRED exception and retry in RecoverableZooKeeper (Allan Yang)
Date Mon, 18 Jul 2016 21:38:04 GMT
Repository: hbase
Updated Branches:
  refs/heads/branch-1.1 d10e0e134 -> 255df6f36


HBASE-16238 It's useless to catch SESSIONEXPIRED exception and retry in RecoverableZooKeeper
(Allan Yang)


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/255df6f3
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/255df6f3
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/255df6f3

Branch: refs/heads/branch-1.1
Commit: 255df6f36d344c853ede37ef5594e65406a0de27
Parents: d10e0e1
Author: stack <stack@apache.org>
Authored: Mon Jul 18 14:36:11 2016 -0700
Committer: stack <stack@apache.org>
Committed: Mon Jul 18 14:37:58 2016 -0700

----------------------------------------------------------------------
 .../hadoop/hbase/zookeeper/RecoverableZooKeeper.java   | 13 -------------
 1 file changed, 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/255df6f3/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
index 11ed8d7..52066c5 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
@@ -189,7 +189,6 @@ public class RecoverableZooKeeper {
               throw e;
 
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "delete");
               break;
@@ -222,7 +221,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "exists");
               break;
@@ -254,7 +252,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "exists");
               break;
@@ -296,7 +293,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "getChildren");
               break;
@@ -328,7 +324,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "getChildren");
               break;
@@ -361,7 +356,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "getData");
               break;
@@ -394,7 +388,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "getData");
               break;
@@ -430,7 +423,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "setData");
               break;
@@ -478,7 +470,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "getAcl");
               break;
@@ -510,7 +501,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "setAcl");
               break;
@@ -595,7 +585,6 @@ public class RecoverableZooKeeper {
             throw e;
 
           case CONNECTIONLOSS:
-          case SESSIONEXPIRED:
           case OPERATIONTIMEOUT:
             retryOrThrow(retryCounter, e, "create");
             break;
@@ -629,7 +618,6 @@ public class RecoverableZooKeeper {
       } catch (KeeperException e) {
         switch (e.code()) {
           case CONNECTIONLOSS:
-          case SESSIONEXPIRED:
           case OPERATIONTIMEOUT:
             retryOrThrow(retryCounter, e, "create");
             break;
@@ -685,7 +673,6 @@ public class RecoverableZooKeeper {
         } catch (KeeperException e) {
           switch (e.code()) {
             case CONNECTIONLOSS:
-            case SESSIONEXPIRED:
             case OPERATIONTIMEOUT:
               retryOrThrow(retryCounter, e, "multi");
               break;


Mime
View raw message