Author: tedyu
Date: Wed Feb 29 14:27:42 2012
New Revision: 1295128

URL: http://svn.apache.org/viewvc?rev=1295128&view=rev
Log:
HBASE-5490  Move the enum RS_ZK_REGION_FAILED_OPEN to the last of the enum list 
in 0.90 EventHandler
               (Ram)

Modified:
    hbase/branches/0.90/CHANGES.txt
    
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java

Modified: hbase/branches/0.90/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.90/CHANGES.txt?rev=1295128&r1=1295127&r2=1295128&view=diff
==============================================================================
--- hbase/branches/0.90/CHANGES.txt (original)
+++ hbase/branches/0.90/CHANGES.txt Wed Feb 29 14:27:42 2012
@@ -52,6 +52,8 @@ Release 0.90.6 - Unreleased
    HBASE-5379  Backport HBASE-4287 to 0.90 - If region opening fails, try to 
transition region back to
                "offline" in ZK
    HBASE-5396  Handle the regions in regionPlans while processing 
ServerShutdownHandler(Jieshan)
+   HBASE-5490  Move the enum RS_ZK_REGION_FAILED_OPEN to the last of the enum 
list in 0.90 EventHandler
+               (Ram)
 
   IMPROVEMENT
    HBASE-5102  Change the default value of the property 
"hbase.connection.per.config" to false in

Modified: 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java
URL: 
http://svn.apache.org/viewvc/hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java?rev=1295128&r1=1295127&r2=1295128&view=diff
==============================================================================
--- 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java
 (original)
+++ 
hbase/branches/0.90/src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java
 Wed Feb 29 14:27:42 2012
@@ -107,7 +107,6 @@ public abstract class EventHandler imple
     RS_ZK_REGION_CLOSED       (2),   // RS has finished closing a region
     RS_ZK_REGION_OPENING      (3),   // RS is in process of opening a region
     RS_ZK_REGION_OPENED       (4),   // RS has finished opening a region
-    RS_ZK_REGION_FAILED_OPEN  (5),   // RS failed to open a region
 
     // Messages originating from Master to RS
     M_RS_OPEN_REGION          (20),  // Master asking RS to open a region
@@ -132,7 +131,13 @@ public abstract class EventHandler imple
 
     // Master controlled events to be executed on the master
     M_SERVER_SHUTDOWN         (70),  // Master is processing shutdown of a RS
-    M_META_SERVER_SHUTDOWN    (72);  // Master is processing shutdown of RS 
hosting a meta region (-ROOT- or .META.).
+    M_META_SERVER_SHUTDOWN(72), // Master is processing shutdown of RS hosting 
a
+                                // meta region (-ROOT- or .META.).
+    
+    // Moving to the last to avoid rolling restart issues
+    // The integer passed in the constructor is not getting used while
+    // serialization.
+    RS_ZK_REGION_FAILED_OPEN(73); // RS failed to open a region
 
     /**
      * Constructor


Reply via email to