[jira] [Commented] (HBASE-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626853#comment-13626853 ] Sergey Shelukhin commented on HBASE-8219: - The merge of the merges has been merged :) > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Fix For: 0.98.0, 0.95.2 > > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626781#comment-13626781 ] Hudson commented on HBASE-8219: --- Integrated in HBase-TRUNK #4046 (See [https://builds.apache.org/job/HBase-TRUNK/4046/]) HBASE-8219 Align Offline Merge with Online Merge (Revision 1465943) Result = FAILURE zjushch : Files : * /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java * /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionMergeTransaction.java > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Fix For: 0.98.0, 0.95.2 > > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626739#comment-13626739 ] Hudson commented on HBASE-8219: --- Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #489 (See [https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/489/]) HBASE-8219 Align Offline Merge with Online Merge (Revision 1465943) Result = FAILURE zjushch : Files : * /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java * /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionMergeTransaction.java > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Fix For: 0.98.0, 0.95.2 > > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626643#comment-13626643 ] Hudson commented on HBASE-8219: --- Integrated in hbase-0.95 #135 (See [https://builds.apache.org/job/hbase-0.95/135/]) HBASE-8219 Align Offline Merge with Online Merge (Revision 1465953) Result = FAILURE zjushch : Files : * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionMergeTransaction.java > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Fix For: 0.98.0, 0.95.2 > > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626597#comment-13626597 ] Hudson commented on HBASE-8219: --- Integrated in hbase-0.95-on-hadoop2 #62 (See [https://builds.apache.org/job/hbase-0.95-on-hadoop2/62/]) HBASE-8219 Align Offline Merge with Online Merge (Revision 1465953) Result = FAILURE zjushch : Files : * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java * /hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionMergeTransaction.java > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Fix For: 0.98.0, 0.95.2 > > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13626426#comment-13626426 ] chunhui shen commented on HBASE-8219: - Committed to trunk and 0.95, Thanks for the review,matteo, ted > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13625279#comment-13625279 ] Matteo Bertozzi commented on HBASE-8219: +1 on patch v3 > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13624579#comment-13624579 ] chunhui shen commented on HBASE-8219: - [~mbertozzi] What do you think about the latest patch ? > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch, > hbase-8219v3.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13619412#comment-13619412 ] chunhui shen commented on HBASE-8219: - bq.the other two regions are removed by using the archiver,are they empty after the compaction? what does the 'empty' mean? After the compaction, the two regions' storefiles are still in their directory. Then archiving these files (may be used for snapshot) by archiver > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13619075#comment-13619075 ] Matteo Bertozzi commented on HBASE-8219: A compaction is called on the merge region, this means that after that call the region will not have reference files, but the real data. right? so at this point the other two regions are removed by using the archiver, are they empty after the compaction? they should, right... maybe add an assert there. if what I have said above is true, I'm +1 on the patch (with the assert if you want) > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13618867#comment-13618867 ] Ted Yu commented on HBASE-8219: --- +1 from me. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13618616#comment-13618616 ] Hadoop QA commented on HBASE-8219: -- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12576331/hbase-8219v2.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:green}+1 site{color}. The mvn site goal succeeds with this patch. {color:green}+1 core tests{color}. The patch passed unit tests in . Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5077//console This message is automatically generated. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13618597#comment-13618597 ] chunhui shen commented on HBASE-8219: - bq.The second and third calls wouldn't have effect because server is null.So if we expose createMergedRegion() as public method, we can call it directly. merged.openHRegion(reporter) is also called in RegionMergeTransaction#openMergedRegion. bq.Maybe we can also implement hbase shell bindings for online one. Shell command of online region is already done(HBASE-8189) IMHO, 1.We should fix the snapshot break caused by offline merge now 2.Do some easy alignment about the code of online merge and offline merge, since we plan to remove offline merge in a futuer moment > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch, hbase-8219v2.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617543#comment-13617543 ] Hadoop QA commented on HBASE-8219: -- {color:red}-1 overall{color}. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12576094/hbase-8219v1.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 site{color}. The patch appears to cause mvn site goal to fail. {color:red}-1 core tests{color}. The patch failed these unit tests: org.apache.hadoop.hbase.master.TestSplitLogManager Test results: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-protocol.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-client.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-examples.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop1-compat.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-server.html Findbugs warnings: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//artifact/trunk/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/5046//console This message is automatically generated. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617529#comment-13617529 ] Enis Soztutar commented on HBASE-8219: -- BTW, in this issue or a follow up, let's merge org.apache.hadoop.hbase.util.Merge, org.apache.hadoop.hbase.util.HMerge, and the recent online merge bits. Maybe we can also implement hbase shell bindings for online one. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617526#comment-13617526 ] Jean-Marc Spaggiari commented on HBASE-8219: [~sershe] I don't think it's a legal condition, but it might allow you to merge 2 regions having boundaries issues... And then create a cleaned one. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617524#comment-13617524 ] Enis Soztutar commented on HBASE-8219: -- Some dev@ discussions for this was at: http://search-hadoop.com/m/j2Szb1U3Bzf/online+merge&subj=Re+How+to+replace+MetaUtils+ScannerListener+ I agree that there should be one tool called Merge, which would work in online mode or offline mode. I was suggesting to get rid of offline one, because on the longer term, it does not make sense to have offline split / merge code. But let's keep the offline merge for now, and reevaluate when the time comes. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617520#comment-13617520 ] Sergey Shelukhin commented on HBASE-8219: - bq. (It could be caused if one merging region's range contains another's) Is it a legal condition? > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617493#comment-13617493 ] Ted Yu commented on HBASE-8219: --- {code} + throw new IOException("Unable to merge region " + a + " and " + b); {code} 'merge region' -> 'merge regions' {code} + dstRegion = rmt.execute(null, null); {code} Looking into execute() method: {code} HRegion mergedRegion = createMergedRegion(server, services); openMergedRegion(server, services, mergedRegion); transitionZKNode(server, services); return mergedRegion; {code} The second and third calls wouldn't have effect because server is null. So if we expose createMergedRegion() as public method, we can call it directly. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi >Assignee: chunhui shen > Attachments: hbase-8219v1.patch > > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617452#comment-13617452 ] Jonathan Hsieh commented on HBASE-8219: --- I like the idea of having them share common code, and don't like the idea of removing offline at the moment. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617381#comment-13617381 ] Jean-Marc Spaggiari commented on HBASE-8219: Alligning the 2 will fix the snapshots break, no? > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617364#comment-13617364 ] chunhui shen commented on HBASE-8219: - Alignment or just fixing the snapshots break? I'm glad to make a patch for this if you trust me. > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617326#comment-13617326 ] Jean-Marc Spaggiari commented on HBASE-8219: [~zjushch] are you going to take care of that? > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617299#comment-13617299 ] chunhui shen commented on HBASE-8219: - +1 on making offline merge not break snapshots, it would change a little code which we only need create reference files instead of moving files in offline merge > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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-8219) Align Offline Merge with Online Merge
[ https://issues.apache.org/jira/browse/HBASE-8219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13617285#comment-13617285 ] Matteo Bertozzi commented on HBASE-8219: I missed the remove discussion, but I'm +1 on it since the offline requires the cluster to be down. Should we consider this jira as fixing the offline merge in 0.94 just to don't break snapshots? > Align Offline Merge with Online Merge > - > > Key: HBASE-8219 > URL: https://issues.apache.org/jira/browse/HBASE-8219 > Project: HBase > Issue Type: Task > Components: regionserver >Affects Versions: 0.95.0 >Reporter: Matteo Bertozzi > > After HBASE-7403 we now have two different tools for online and offline > merge, and the result produced by the two are different. (the online one > works with snapshots, the offline not) > We should remove the offline one, or align it to the online code. > Most of the offline code in HRegion.merge() can be replaced with the one in > RegionMergeTransaction, used by the online version. -- 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