[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-02-04 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13570998#comment-13570998
 ] 

Hudson commented on HBASE-7506:
---

Integrated in HBase-0.94-security-on-Hadoop-23 #11 (See 
[https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/11/])
HBASE-7506 Judgment of carrying ROOT/META will become wrong when expiring 
server (Chunhui) (Revision 1431903)

 Result = FAILURE
zjushch : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-94.patch, 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-15 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13554225#comment-13554225
 ] 

Hudson commented on HBASE-7506:
---

Integrated in HBase-0.94-security #95 (See 
[https://builds.apache.org/job/HBase-0.94-security/95/])
HBASE-7506 Judgment of carrying ROOT/META will become wrong when expiring 
server (Chunhui) (Revision 1431903)

 Result = SUCCESS
zjushch : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-94.patch, 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-11 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13550960#comment-13550960
 ] 

Hudson commented on HBASE-7506:
---

Integrated in HBase-0.94 #723 (See 
[https://builds.apache.org/job/HBase-0.94/723/])
HBASE-7506 Judgment of carrying ROOT/META will become wrong when expiring 
server (Chunhui) (Revision 1431903)

 Result = SUCCESS
zjushch : 
Files : 
* 
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-94.patch, 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-10 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13549435#comment-13549435
 ] 

Hadoop QA commented on HBASE-7506:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12564138/7506-trunkv1.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:green}+1 hadoop2.0{color}.  The patch compiles against the hadoop 
2.0 profile.

{color:green}+1 javadoc{color}.  The javadoc tool did not generate any 
warning messages.

{color:green}+1 javac{color}.  The applied patch does not increase the 
total number of javac compiler warnings.

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 1.3.9) warnings.

{color:green}+1 release audit{color}.  The applied patch does not increase 
the total number of release audit warnings.

{color:green}+1 lineLengths{color}.  The patch does not introduce lines 
longer than 100

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.regionserver.TestSplitTransaction

 {color:red}-1 core zombie tests{color}.  There are 1 zombie test(s):   
at 
org.apache.hadoop.hdfs.server.balancer.TestBalancerWithNodeGroup.testBalancerWithRackLocality(TestBalancerWithNodeGroup.java:220)

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3958//console

This message is automatically generated.

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-10 Thread stack (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13550845#comment-13550845
 ] 

stack commented on HBASE-7506:
--

Are you going to commit [~zjushch]?  You have two +1s.

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-10 Thread chunhui shen (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13550849#comment-13550849
 ] 

chunhui shen commented on HBASE-7506:
-

Patch v1 integrated to trunk, 0.94 branch

Thanks for the review ram,jimmy, ted

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-10 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13550857#comment-13550857
 ] 

Hadoop QA commented on HBASE-7506:
--

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12564363/7506-94.patch
  against trunk revision .

{color:green}+1 @author{color}.  The patch does not contain any @author 
tags.

{color:red}-1 tests included{color}.  The patch doesn't appear to include 
any new or modified tests.
Please justify why no new tests are needed for this 
patch.
Also please list what manual steps were performed to 
verify this patch.

{color:red}-1 patch{color}.  The patch command could not apply the patch.

Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/3977//console

This message is automatically generated.

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-94.patch, 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-10 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13550910#comment-13550910
 ] 

Hudson commented on HBASE-7506:
---

Integrated in HBase-TRUNK #3728 (See 
[https://builds.apache.org/job/HBase-TRUNK/3728/])
HBASE-7506 Judgment of carrying ROOT/META will become wrong when expiring 
server (Chunhui) (Revision 1431901)

 Result = FAILURE
zjushch : 
Files : 
* 
/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
* 
/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/master/handler/ServerShutdownHandler.java


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0, 0.94.5

 Attachments: 7506-94.patch, 7506-trunk v1.patch, 7506-trunkv1.patch, 
 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-09 Thread ramkrishna.s.vasudevan (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13549339#comment-13549339
 ] 

ramkrishna.s.vasudevan commented on HBASE-7506:
---

In v1 patch we are once again verifying if the ROOT and META assignment is what 
we thought.  Seems simple.
+1 on v1.

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch, 7506-trunkv2.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-08 Thread Jimmy Xiang (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13547056#comment-13547056
 ] 

Jimmy Xiang commented on HBASE-7506:


The only reason we have MetaServerShutdownHandler is because of the event type: 
EventType.M_META_SERVER_SHUTDOWN. ServerShutdownHandler already has an event 
type member variable.

If isCarryingRoot/isCarryingMeta is useful, it makes sense to have 
MetaServerShutdownHandler.  Now since they are not used any more, why do we 
still need this class?

bq. ROOT won't be in RIT, because we offline it

Make sense.  That means we should not verify any more, right?

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-08 Thread Ted Yu (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13547064#comment-13547064
 ] 

Ted Yu commented on HBASE-7506:
---

@Jimmy:
w.r.t. MetaServerShutdownHandler, see point 1 from 
https://issues.apache.org/jira/browse/HBASE-3809?focusedCommentId=13546633page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13546633

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-08 Thread Jimmy Xiang (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13547182#comment-13547182
 ] 

Jimmy Xiang commented on HBASE-7506:


@Ted, I read point 1 of HBASE-3809. Different thread pool is used just based on 
the EventType, not class name.  So it should be fine.

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-08 Thread chunhui shen (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13547561#comment-13547561
 ] 

chunhui shen commented on HBASE-7506:
-

bq.That means we should not verify any more, right?
We should also verify for better, see HBASE-7504


I think we could remove MetaServerShutdownHandler class, if we pass the 
carryingRoot,carryingMeta to ServerShutdownHandler. Like the following
{code}
ServerShutdownHandler(final Server server, final MasterServices services,
  final DeadServer deadServers, final ServerName serverName,
  final boolean shouldSplitHlog,final boolean carryingRoot, final boolean 
carryingMeta){
super(server, 
(carryingRoot||carryingMeta)?EventType.M_META_SERVER_SHUTDOWN:EventType.M_SERVER_SHUTDOWN);
{code}


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-07 Thread ramkrishna.s.vasudevan (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13546048#comment-13546048
 ] 

ramkrishna.s.vasudevan commented on HBASE-7506:
---

You got this in trunk?  Am not able to visualise the problem due to not so good 
grip on the Trunk code base wrt recent changes.
May be Jimmy can look this.
Will try to look into it once again later tomorrow.
[~jxiang]
Want to take a look at this?

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-07 Thread Jimmy Xiang (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13546123#comment-13546123
 ] 

Jimmy Xiang commented on HBASE-7506:


This is subtle. This part is not touched wrt recent changes. I agree it is 
possible root/meta could be assigning to the dead rs, which is processing by 
SSH.
The existing code assigns root/meta again is not efficient, but won't cause any 
problem in such a case.

It makes sense to check it again. However, if they are still RIT at this 
re-checking moment, it is probably not efficient to re-assign them than wait, 
right?

So I think we can do the following:

1. remove class MetaServerShutdownHandler;
2. in ServerShutdownHandler, make isCarryingRoot and isCarryingMeta local 
variables to process method, dynamically set it based on AM, as in Chunhui's 
patch;
3. in verifyAndAssignRootWithRetries, check if root is RIT, if so, should we 
time it out and wait, or just force-reassgin?
4. Can we have a similar verifyAndAssignMetaWithRetries in SSH?  If meta can be 
verified, no need to re-assign?

[~ram_krish], what do you think?

Adding [~nkeywal] too.


 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-07 Thread chunhui shen (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13546614#comment-13546614
 ] 

chunhui shen commented on HBASE-7506:
-

[~jxiang]
bq.3.in verifyAndAssignRootWithRetries, check if root is RIT
-ROOT- won't be in RIT, because we offline it
{code}
if (isCarryingRoot()) { // -ROOT-
LOG.info(Server  + serverName +
 was carrying ROOT. Trying to assign.);
this.services.getAssignmentManager().
  regionOffline(HRegionInfo.ROOT_REGIONINFO);
{code}

bq.1. remove class MetaServerShutdownHandler;
MetaServerShutdownHandler and ServerShutdownHandler will be submit to different 
ExecutorService, that's why we using MetaServerShutdownHandler 



 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (HBASE-7506) Judgment of carrying ROOT/META will become wrong when expiring server

2013-01-07 Thread Ted Yu (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-7506?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13546621#comment-13546621
 ] 

Ted Yu commented on HBASE-7506:
---

I think we should keep MetaServerShutdownHandler.
Take a look at HBASE-3809

 Judgment of carrying ROOT/META will become wrong when expiring server
 -

 Key: HBASE-7506
 URL: https://issues.apache.org/jira/browse/HBASE-7506
 Project: HBase
  Issue Type: Bug
Affects Versions: 0.94.3
Reporter: chunhui shen
Assignee: chunhui shen
 Fix For: 0.96.0

 Attachments: 7506-trunk v1.patch


 We will check whether server carrying ROOT/META when expiring the server.
 See ServerManager#expireServer.
 If the dead server carrying META, we assign meta directly in the process of 
 ServerShutdownHandler.
 If the dead server carrying ROOT, we will offline ROOT and then 
 verifyAndAssignRootWithRetries()
 How judgement of carrtying ROOT/META become wrong?
 If region is in RIT, and isCarryingRegion() return true after addressing from 
 zk.
 However, once RIT time out(could be caused by this.allRegionServersOffline  
 !noRSAvailable, see AssignmentManager#TimeoutMonitor)   and we assign it to 
 otherwhere, this judgement become wrong.
 See AssignmentManager#isCarryingRegion for details
 With the wrong judgement of carrtying ROOT/META, we would assign ROOT/META 
 twice.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira