You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2016/07/18 21:36:16 UTC

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

Repository: hbase
Updated Branches:
  refs/heads/master 515c499f9 -> c9925bcec


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/c9925bce
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/c9925bce
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/c9925bce

Branch: refs/heads/master
Commit: c9925bcec1f5cd427c4a875dba64b92e3dafcc1e
Parents: 515c499
Author: stack <st...@apache.org>
Authored: Mon Jul 18 14:36:11 2016 -0700
Committer: stack <st...@apache.org>
Committed: Mon Jul 18 14:36:11 2016 -0700

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


http://git-wip-us.apache.org/repos/asf/hbase/blob/c9925bce/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 e3c3b8c..e8db4da 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;