[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-22 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137340#comment-16137340
 ] 

Hudson commented on HBASE-18103:


FAILURE: Integrated in Jenkins build HBASE-14070.HLC #263 (See 
[https://builds.apache.org/job/HBASE-14070.HLC/263/])
HBASE-18103 [AMv2] Changed master to throw YouAreDeadException when it (stack: 
rev 5895538a32d6bd2d7153e38304523caf6ea7579c)
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/MockMasterServices.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
* (add) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/TestRogueRSAssignment.java


> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>Priority: Critical
> Fix For: 2.0.0
>
> Attachments: hbase-18103.master.001.patch, 
> hbase-18103.master.001.patch
>
>
> If a RS is slow to open a Region, the Master will give the Region to another 
> to open it (In this case, was a massive set of edits to process and a load of 
> StoreFiles to open...). Should the original RS succeed with its open 
> eventually, on reporting the master the successful open, the Master currently 
> kills the RS because the region is supposed to be elsewhere.
> This is an easy fix.
> The RS does not fully open a Region until Master gives it the go so just 
> close the region if master rejects the open
> See '6.1.1 If Master gives Region to another to Open, old RS will be kill 
> itself on reject by Master; easy fix!' in 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.qtfojp9774h



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16136348#comment-16136348
 ] 

Hudson commented on HBASE-18103:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3574 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3574/])
HBASE-18103 [AMv2] Changed master to throw YouAreDeadException when it (stack: 
rev 5895538a32d6bd2d7153e38304523caf6ea7579c)
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/MockMasterServices.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
* (add) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/TestRogueRSAssignment.java


> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>Priority: Critical
> Fix For: 2.0.0
>
> Attachments: hbase-18103.master.001.patch, 
> hbase-18103.master.001.patch
>
>
> If a RS is slow to open a Region, the Master will give the Region to another 
> to open it (In this case, was a massive set of edits to process and a load of 
> StoreFiles to open...). Should the original RS succeed with its open 
> eventually, on reporting the master the successful open, the Master currently 
> kills the RS because the region is supposed to be elsewhere.
> This is an easy fix.
> The RS does not fully open a Region until Master gives it the go so just 
> close the region if master rejects the open
> See '6.1.1 If Master gives Region to another to Open, old RS will be kill 
> itself on reject by Master; easy fix!' in 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.qtfojp9774h



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16136317#comment-16136317
 ] 

Hudson commented on HBASE-18103:


FAILURE: Integrated in Jenkins build HBase-2.0 #373 (See 
[https://builds.apache.org/job/HBase-2.0/373/])
HBASE-18103 [AMv2] Changed master to throw YouAreDeadException when it (stack: 
rev b059d8602d465e7722fcbf27bc775e1663e699c7)
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/MockMasterServices.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java
* (add) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/assignment/TestRogueRSAssignment.java


> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>Priority: Critical
> Fix For: 2.0.0
>
> Attachments: hbase-18103.master.001.patch, 
> hbase-18103.master.001.patch
>
>
> If a RS is slow to open a Region, the Master will give the Region to another 
> to open it (In this case, was a massive set of edits to process and a load of 
> StoreFiles to open...). Should the original RS succeed with its open 
> eventually, on reporting the master the successful open, the Master currently 
> kills the RS because the region is supposed to be elsewhere.
> This is an easy fix.
> The RS does not fully open a Region until Master gives it the go so just 
> close the region if master rejects the open
> See '6.1.1 If Master gives Region to another to Open, old RS will be kill 
> itself on reject by Master; easy fix!' in 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.qtfojp9774h



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16136055#comment-16136055
 ] 

Hadoop QA commented on HBASE-18103:
---

| (/) *{color:green}+1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
19s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
 3s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
35s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
42s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  2m 
50s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
26s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
35s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
35s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
28m 40s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  3m  
7s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
25s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}110m 
21s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
18s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}153m 32s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:bdc94b1 |
| JIRA Issue | HBASE-18103 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12882973/hbase-18103.master.001.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  findbugs  hadoopcheck  
hbaseanti  checkstyle  compile  |
| uname | Linux 9097bf787d73 4.4.0-43-generic #63-Ubuntu SMP Wed Oct 12 
13:48:03 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / 7b3291f |
| Default Java | 1.8.0_144 |
| findbugs | v3.1.0-RC3 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8207/testReport/ |
| modules | C: hbase-server U: hbase-server |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8207/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>   

[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread Hadoop QA (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16135789#comment-16135789
 ] 

Hadoop QA commented on HBASE-18103:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
17s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 2 new or modified test 
files. {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
24s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
42s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
43s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
13s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  2m 
47s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
25s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  0m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  0m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  0m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  0m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
28m 15s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  2m 
58s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m 
26s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 67m 40s{color} 
| {color:red} hbase-server in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
37s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}110m 56s{color} | 
{color:black} {color} |
\\
\\
|| Reason || Tests ||
| Timed out junit tests | 
org.apache.hadoop.hbase.master.procedure.TestDisableTableProcedure |
|   | org.apache.hadoop.hbase.regionserver.wal.TestSecureWALReplay |
|   | org.apache.hadoop.hbase.replication.TestReplicationKillSlaveRS |
|   | org.apache.hadoop.hbase.master.procedure.TestDeleteTableProcedure |
|   | org.apache.hadoop.hbase.master.procedure.TestEnableTableProcedure |
|   | org.apache.hadoop.hbase.master.procedure.TestCreateTableProcedure |
|   | org.apache.hadoop.hbase.master.procedure.TestModifyTableProcedure |
|   | org.apache.hadoop.hbase.master.procedure.TestServerCrashProcedure |
|   | org.apache.hadoop.hbase.mapreduce.TestWALPlayer |
|   | org.apache.hadoop.hbase.regionserver.TestSplitLogWorker |
|   | org.apache.hadoop.hbase.regionserver.wal.TestAsyncWALReplay |
|   | org.apache.hadoop.hbase.mapreduce.TestTableInputFormat |
|   | org.apache.hadoop.hbase.client.TestSnapshotCloneIndependence |
|   | org.apache.hadoop.hbase.regionserver.compactions.TestFIFOCompactionPolicy 
|
|   | org.apache.hadoop.hbase.master.TestGetLastFlushedSequenceId |
|   | org.apache.hadoop.hbase.client.TestAsyncTableScanAll |
|   | org.apache.hadoop.hbase.client.TestMultipleTimestamps |
|   | org.apache.hadoop.hbase.client.TestFromClientSide |
|   | org.apache.hadoop.hbase.snapshot.TestSnapshotClientRetries |
|   | org.apache.hadoop.hbase.master.balancer.TestStochasticLoadBalancer2 |
|   | org.apache.hadoop.hbase.regionserver.TestRemoveRegionMetrics |
|   | 

[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread stack (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16135595#comment-16135595
 ] 

stack commented on HBASE-18103:
---

Looks good to me. Will wait on hadoopqa. On commit I'll add a note on why the 
exception is thrown... that we consider the RS from left-field rogue.

> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>Priority: Critical
> Fix For: 2.0.0
>
> Attachments: hbase-18103.master.001.patch
>
>
> If a RS is slow to open a Region, the Master will give the Region to another 
> to open it (In this case, was a massive set of edits to process and a load of 
> StoreFiles to open...). Should the original RS succeed with its open 
> eventually, on reporting the master the successful open, the Master currently 
> kills the RS because the region is supposed to be elsewhere.
> This is an easy fix.
> The RS does not fully open a Region until Master gives it the go so just 
> close the region if master rejects the open
> See '6.1.1 If Master gives Region to another to Open, old RS will be kill 
> itself on reject by Master; easy fix!' in 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.qtfojp9774h



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (HBASE-18103) [AMv2] If Master gives OPEN to another, if original eventually succeeds, Master will kill it

2017-08-21 Thread Umesh Agashe (JIRA)

[ 
https://issues.apache.org/jira/browse/HBASE-18103?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16135588#comment-16135588
 ] 

Umesh Agashe commented on HBASE-18103:
--

Modified the code to throw YouAreDeadException() for RS report with incorrect 
assignment and added UT to test this.

> [AMv2] If Master gives OPEN to another, if original eventually succeeds, 
> Master will kill it
> 
>
> Key: HBASE-18103
> URL: https://issues.apache.org/jira/browse/HBASE-18103
> Project: HBase
>  Issue Type: Sub-task
>  Components: master, proc-v2
>Reporter: stack
>Assignee: Umesh Agashe
>Priority: Critical
> Fix For: 2.0.0
>
> Attachments: hbase-18103.master.001.patch
>
>
> If a RS is slow to open a Region, the Master will give the Region to another 
> to open it (In this case, was a massive set of edits to process and a load of 
> StoreFiles to open...). Should the original RS succeed with its open 
> eventually, on reporting the master the successful open, the Master currently 
> kills the RS because the region is supposed to be elsewhere.
> This is an easy fix.
> The RS does not fully open a Region until Master gives it the go so just 
> close the region if master rejects the open
> See '6.1.1 If Master gives Region to another to Open, old RS will be kill 
> itself on reject by Master; easy fix!' in 
> https://docs.google.com/document/d/1eVKa7FHdeoJ1-9o8yZcOTAQbv0u0bblBlCCzVSIn69g/edit#heading=h.qtfojp9774h



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)