[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread stack (JIRA)

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

stack commented on HBASE-11632:
---

+1

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread Andrew Purtell (JIRA)

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

Andrew Purtell commented on HBASE-11632:


+1 for 0.98

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.4
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11632:


FAILURE: Integrated in HBase-0.98-on-Hadoop-1.1 #407 (See 
[https://builds.apache.org/job/HBase-0.98-on-Hadoop-1.1/407/])
HBASE-11632 Region split needs to clear force split flag at the end of 
SplitRequest run (Jerry He) (tedyu: rev 
f2fef2d39a8c69c56ed4ca9c0a52b86d7eb2d03a)
* hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestStore.java
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SplitRequest.java


 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.4
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Fix For: 0.99.0, 0.98.5, 2.0.0

 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch, 
 HBASE-11632-v2-0.98.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11632:


SUCCESS: Integrated in HBase-0.98 #432 (See 
[https://builds.apache.org/job/HBase-0.98/432/])
HBASE-11632 Region split needs to clear force split flag at the end of 
SplitRequest run (Jerry He) (tedyu: rev 
f2fef2d39a8c69c56ed4ca9c0a52b86d7eb2d03a)
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SplitRequest.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestStore.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java


 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.4
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Fix For: 0.99.0, 0.98.5, 2.0.0

 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch, 
 HBASE-11632-v2-0.98.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11632:


FAILURE: Integrated in HBase-1.0 #82 (See 
[https://builds.apache.org/job/HBase-1.0/82/])
HBASE-11632 Region split needs to clear force split flag at the end of 
SplitRequest run (Jerry He) (tedyu: rev 
9db8ecba746498573e9885d86f8be6bbdb77f78a)
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SplitRequest.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestStore.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java


 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.4
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Fix For: 0.99.0, 0.98.5, 2.0.0

 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch, 
 HBASE-11632-v2-0.98.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-08-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-11632:


FAILURE: Integrated in HBase-TRUNK #5364 (See 
[https://builds.apache.org/job/HBase-TRUNK/5364/])
HBASE-11632 Region split needs to clear force split flag at the end of 
SplitRequest run (Jerry He) (tedyu: rev 
eb79a4358029653817ddf8bee0dde392c3368ecb)
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/SplitRequest.java
* 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/CompactSplitThread.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestStore.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegion.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java


 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.4
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Fix For: 0.99.0, 0.98.5, 2.0.0

 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch, 
 HBASE-11632-v2-0.98.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-07-31 Thread Ted Yu (JIRA)

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

Ted Yu commented on HBASE-11632:


lgtm

Is it hard to come up with a test corresponding to the scenario above ?

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-07-31 Thread Jerry He (JIRA)

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

Jerry He commented on HBASE-11632:
--

Hi, [~yuzhih...@gmail.com]
Let me try.  Not easy for exact scenario :-)


 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-07-31 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-11632:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  
http://issues.apache.org/jira/secure/attachment/12658974/HBASE-11632-master.patch
  against trunk revision .
  ATTACHMENT ID: 12658974

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

{color:green}+1 tests included{color}.  The patch appears to include 3 new 
or modified tests.

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

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

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

{color:green}+1 findbugs{color}.  The patch does not introduce any new 
Findbugs (version 2.0.3) 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:green}+1 site{color}.  The mvn site goal succeeds with this patch.

 {color:red}-1 core tests{color}.  The patch failed these unit tests:
   org.apache.hadoop.hbase.client.TestReplicasClient
  org.apache.hadoop.hbase.master.TestRestartCluster
  
org.apache.hadoop.hbase.regionserver.TestEndToEndSplitTransaction
  org.apache.hadoop.hbase.TestRegionRebalancing
  org.apache.hadoop.hbase.regionserver.TestRegionReplicas
  
org.apache.hadoop.hbase.master.TestMasterOperationsForRegionReplicas
  org.apache.hadoop.hbase.TestIOFencing

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Console output: 
https://builds.apache.org/job/PreCommit-HBASE-Build/10247//console

This message is automatically generated.

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-07-31 Thread Jerry He (JIRA)

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

Jerry He commented on HBASE-11632:
--

The test case failures from the previous QA run don't seem to be related.

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (HBASE-11632) Region split needs to clear force split flag at the end of SplitRequest run

2014-07-31 Thread Jerry He (JIRA)

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

Jerry He commented on HBASE-11632:
--

Attached v2 that added another catch to clear the force split flag, and added a 
test case.

 Region split needs to clear force split flag at the end of SplitRequest run
 ---

 Key: HBASE-11632
 URL: https://issues.apache.org/jira/browse/HBASE-11632
 Project: HBase
  Issue Type: Bug
  Components: regionserver
Affects Versions: 0.99.0, 0.98.5
Reporter: Jerry He
Assignee: Jerry He
Priority: Minor
 Attachments: HBASE-11632-master-v2.patch, HBASE-11632-master.patch


 force split flag is used when a user requests region split and specifies a 
 split key.
 {code}
 region.forceSplit(splitPoint);
 {code}
 We need to clear force split flag at the end of SplitRequest run. If the 
 region split is successful, this parent region will go away and there is no 
 need to clear this flag.  But we will have problem when the split is not 
 successful or rejected.  Here is the case:
 1. User requested region split at rownkey-n.
 2. We set forceSplit in the region with rownkey-n.
 3. The split request got rejected because the region had references.
 4. After a while, there was a system request split for the same region.
 5. The region would use forceSplit and rownkey-n to split.  What should have 
 happened is that the region split is done based on system split policy.



--
This message was sent by Atlassian JIRA
(v6.2#6252)