[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Dima Spivak (JIRA)

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

Dima Spivak commented on HBASE-12740:
-

:) I'm convinced. You're awesome, [~jurmous]. Just two minor nits:

- Since poolSize is a constant, we should probably have it be a static final 
int and call it POOL_SIZE to make it stick out.
- When constructing the scheduled thread pool executor, we should pass in 
POOL_SIZE instead of the hardcoded 7 (if I'm not mistaken).

Other than that, great work. +1

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12689556/HBASE-12740-v4.patch
  against master branch at commit c6002ccd3873f3950df50bef7f4ac4c5760d595f.
  ATTACHMENT ID: 12689556

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

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

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

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740.patch, 
 PROFILE_after_patch.png, PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread stack (JIRA)

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

stack commented on HBASE-12740:
---

Pardon me. I applied v3 with [~dimaspivak] items addressed.  Was then trying to 
cherry-pick to branch-1 and beyond but was taking a little while (TestHBaseFsck 
runs are a bit of beast).  I reverted my master work since [~jurmous] showed up 
w/ new version.  Will go forward with v4.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-12740:


SUCCESS: Integrated in HBase-TRUNK #5978 (See 
[https://builds.apache.org/job/HBase-TRUNK/5978/])
HBASE-12740 Improve performance of TestHBaseFsck (stack: rev 
042743fff076a66e25c93fce97ab3e75adb8190d)
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildHole.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java


 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12689560/HBASE-12740-v4.patch
  against master branch at commit 734324e80f44be4d40b1ee4612a838b581d504b2.
  ATTACHMENT ID: 12689560

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

{color:green}+1 tests included{color}.  The patch appears to include 11 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 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{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:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12267//artifact/patchprocess/checkstyle-aggregate.html

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-12740:


SUCCESS: Integrated in HBase-TRUNK #5979 (See 
[https://builds.apache.org/job/HBase-TRUNK/5979/])
Revert HBASE-12740 Improve performance of TestHBaseFsck (stack: rev 
734324e80f44be4d40b1ee4612a838b581d504b2)
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildHole.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java


 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-12740:


SUCCESS: Integrated in HBase-1.0 #621 (See 
[https://builds.apache.org/job/HBase-1.0/621/])
HBASE-12740 Improve performance of TestHBaseFsck (stack: rev 
d940e8f6c48ffafb4406da572c05ee6353d4c28b)
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildHole.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java


 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Fix For: 1.0.0, 2.0.0, 1.1.0

 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-12740:


FAILURE: Integrated in HBase-TRUNK #5980 (See 
[https://builds.apache.org/job/HBase-TRUNK/5980/])
HBASE-12740 Improve performance of TestHBaseFsck (stack: rev 
baa2d05374efc0f5b95b8f1f779462038a9f96b2)
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildHole.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java


 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Fix For: 1.0.0, 2.0.0, 1.1.0

 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-30 Thread Hudson (JIRA)

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

Hudson commented on HBASE-12740:


SUCCESS: Integrated in HBase-1.1 #38 (See 
[https://builds.apache.org/job/HBase-1.1/38/])
HBASE-12740 Improve performance of TestHBaseFsck (stack: rev 
ad7d7f59775556157215a710456d8401056165c6)
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/HbckTestingUtil.java
* 
hbase-server/src/test/java/org/apache/hadoop/hbase/util/hbck/TestOfflineMetaRebuildHole.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java


 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Fix For: 1.0.0, 2.0.0, 1.1.0

 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740-v4.patch, HBASE-12740-v4.patch, 
 HBASE-12740.patch, PROFILE_after_patch.png, 
 PROFILE_before_patch_test_fails.png


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-29 Thread Dima Spivak (JIRA)

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

Dima Spivak commented on HBASE-12740:
-

Hey [~jurmous],

Sorry for the delay in getting back to this; busy time with the holidays. I 
finally had a chance to apply your patch to master (thanks again for rebasing 
for me) and, having run it a half dozen times, I can't seem to reproduce the 
speed-up you're seeing on TestHBaseFsck. Running Maven 3.0.4 and Java 1.7u67, I 
get averaged runtimes of about 277 s before the patch is applied and 287 s 
afterwards. Am I missing something?

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-23 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12688843/HBASE-12740-v1.patch
  against master branch at commit 0d5f967c7ff4809ca782f68a9e48896dd04b16f6.
  ATTACHMENT ID: 12688843

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

{color:green}+1 tests included{color}.  The patch appears to include 11 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 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{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:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12205//artifact/patchprocess/checkstyle-aggregate.html

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-23 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12688855/HBASE-12740-v2.patch
  against master branch at commit 0d5f967c7ff4809ca782f68a9e48896dd04b16f6.
  ATTACHMENT ID: 12688855

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

{color:green}+1 tests included{color}.  The patch appears to include 11 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 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{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:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12207//artifact/patchprocess/checkstyle-aggregate.html

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-23 Thread Dima Spivak (JIRA)

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

Dima Spivak commented on HBASE-12740:
-

Jurriaan,

Would you mind posting another patch of this rebased to the latest commit of 
master? I can't apply it cleanly since it collides with your changes in 
HBASE-12736. I'll take one final look around once hadoop-qa runs on that, but 
this looks fantastic. Well done.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-23 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:green}+1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12688938/HBASE-12740-v3.patch
  against master branch at commit 2b090d6832c944ea47f81e1b6db518a74c1149e0.
  ATTACHMENT ID: 12688938

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

{color:green}+1 tests included{color}.  The patch appears to include 11 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 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

{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:green}+1 core tests{color}.  The patch passed unit tests in .

Test results: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//testReport/
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12215//artifact/patchprocess/checkstyle-aggregate.html

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740-v1.patch, HBASE-12740-v2.patch, 
 HBASE-12740-v3.patch, HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-22 Thread Dima Spivak (JIRA)

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

Dima Spivak commented on HBASE-12740:
-

Thanks so much for taking this on, Jurriaan (especially taking a good step in 
moving to the new non H- objects). Looks lovely, with some comments:

- Since I see a lot of 7s and factors of 7, might it be worth defining a 
constant at the top and then just using throughout so that if this needs tuning 
in the future, it can be changed in one place?
- HBaseTestingUtility has its own deleteTable method. Might it be worth 
plugging those in here instead of having this alternative implementation around?
- What was the rationale for having setupTableWithRegionReplica return an 
HTable instance (instead of a plain Table)?

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-22 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-12740:
---

{color:red}-1 overall{color}.  Here are the results of testing the latest 
attachment 
  http://issues.apache.org/jira/secure/attachment/12688662/HBASE-12740.patch
  against master branch at commit 4d53fe5dcd7b3bd8ca59e2a0683697e2cddaecfc.
  ATTACHMENT ID: 12688662

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

{color:green}+1 tests included{color}.  The patch appears to include 4 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 checkstyle{color}.  The applied patch does not increase the 
total number of checkstyle errors

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

{color:red}-1 release audit{color}.  The applied patch generated 1 release 
audit warnings (more than the master's current 0 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/12186//testReport/
Release audit warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/patchReleaseAuditWarnings.txt
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-client.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-annotations.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-thrift.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-server.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop2-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-protocol.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-examples.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-hadoop-compat.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-prefix-tree.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-common.html
Findbugs warnings: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/newPatchFindbugsWarningshbase-rest.html
Checkstyle Errors: 
https://builds.apache.org/job/PreCommit-HBASE-Build/12186//artifact/patchprocess/checkstyle-aggregate.html

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

This message is automatically generated.

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (HBASE-12740) Improve performance of TestHBaseFsck

2014-12-22 Thread stack (JIRA)

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

stack commented on HBASE-12740:
---

The releaseaudit warning is not this patch and has since been fixed. 


Unapproved licenses:

  src/main/docbkx/hbase-default.xml

Patch looks great caveat [~dimaspivak] feedback

 Improve performance of TestHBaseFsck
 

 Key: HBASE-12740
 URL: https://issues.apache.org/jira/browse/HBASE-12740
 Project: HBase
  Issue Type: Bug
  Components: util
Reporter: Jurriaan Mous
Assignee: Jurriaan Mous
 Attachments: HBASE-12740.patch


 TestHBaseFsck performs poor on my machine. It crashes because the threads 
 reach the 2000 thread limit on my machine. Looking at the code a lot of 
 optimization is possible and some API calls are used wrong. A lot of Admin 
 instances are created and never closed, lots of Tables are not closed, 
 ThreadPoolExecutors are not shut down and an unlimited thread pool which does 
 not recycle threads.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)