[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-13 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3712 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3712/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
8bbfc836db46336b847fe10d1bd0c9a5da8b2647)
* (edit) hbase-server/pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-13 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-2.0 #511 (See 
[https://builds.apache.org/job/HBase-2.0/511/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
62db6bf1a45aa33138370c744a050d6b448dec4e)
* (edit) hbase-server/pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-06 Thread Umesh Agashe (JIRA)

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

Umesh Agashe commented on HBASE-18723:
--

Thanks [~stack] for fixing this and unblocking HBASE-18674!

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-05 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

Ok. Fixes for  HBASE-18674 are complete. The last addendum done here seems to 
have fixed dependency issues for now. Leaving resolved.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-05 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-2.0 #458 (See 
[https://builds.apache.org/job/HBase-2.0/458/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
48bc132e12431e0369275c2d6a578e46bfd60464)
* (edit) hbase-rest/pom.xml
* (edit) hbase-thrift/pom.xml
* (edit) hbase-server/pom.xml
* (edit) hbase-it/pom.xml
* (edit) pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-04 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3660 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3660/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
52d4b25754a227983170bdbadc477756a639ea8e)
* (edit) pom.xml
* (edit) hbase-rest/pom.xml
* (edit) hbase-thrift/pom.xml
* (edit) hbase-server/pom.xml
* (edit) hbase-it/pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-04 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

The TestCanaryTool was because of HBASE-18736; an addendum just applied should 
fix it. Going to push this second addendum.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-04 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
18s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
38s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
55s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m 
56s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
58s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
44s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
21s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m 
51s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  4m 
51s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  3m 
 2s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m  
7s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
33m 38s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
54s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red} 90m 28s{color} 
| {color:red} hbase-server in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
49s{color} | {color:green} hbase-thrift in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
25s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
46s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}100m 43s{color} 
| {color:red} root in the patch failed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  1m 
12s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}265m 47s{color} | 
{color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | hadoop.hbase.tool.TestCanaryTool |
|   | hadoop.hbase.tool.TestCanaryTool |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=1.12.3 Server=1.12.3 Image:yetus/hbase:47a5614 |
| JIRA Issue | HBASE-18723 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12885281/HBASE-18723.master.005.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  xml  compile  |
| uname | Linux 725601bc1aa5 3.13.0-119-generic #166-Ubuntu SMP Wed May 3 
12:18:55 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / bea5de0 |
| Default Java | 1.8.0_144 |
| unit | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8463/artifact/patchprocess/patch-unit-hbase-server.txt
 |
| unit | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8463/artifact/patchprocess/patch-unit-root.txt
 |
|  Test Results | 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-04 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.005 is addendum2. Adds include of a lib that has jspc in it for the antrun jsp 
precompiles. Also adds to hbase-it a reference to shaded hbase-protocol. See 
HBASE-18674 for the failures that are showing these holes on build box (Cannot 
repro. locally).

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, HBASE-18723.master.005.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-03 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-2.0 #451 (See 
[https://builds.apache.org/job/HBase-2.0/451/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
91ab25b469a441904f883dd8e130af2653a2609d)
* (edit) hbase-mapreduce/pom.xml
* (edit) hbase-backup/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-03 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

Pushed to master and branch-2. Resolving though there might be another cycle of 
cleanup todo  yet.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-03 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3653 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3653/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
0e95a8a0ae24b0d19b391d49794d6716a8e86bcd)
* (edit) hbase-mapreduce/pom.xml
* (edit) hbase-backup/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-03 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
21s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
34s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
45s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m  
7s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
41s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
10s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
18s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
27s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m  
7s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  4m  
7s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m  
5s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
36m 10s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 10m  
1s{color} | {color:green} hbase-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  9m 
44s{color} | {color:green} hbase-backup in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  4m  
3s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}196m 
42s{color} | {color:green} root in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  1m 
 5s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}287m 49s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:47a5614 |
| JIRA Issue | HBASE-18723 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12885116/HBASE-18723.master.004.patch
 |
| Optional Tests |  asflicense  javac  javadoc  unit  xml  compile  |
| uname | Linux 8e549bd38f83 3.13.0-116-generic #163-Ubuntu SMP Fri Mar 31 
14:13:22 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | 
/home/jenkins/jenkins-slave/workspace/PreCommit-HBASE-Build/component/dev-support/hbase-personality.sh
 |
| git revision | master / 7c51d3f |
| Default Java | 1.8.0_144 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8445/testReport/ |
| modules | C: hbase-mapreduce hbase-backup hbase-rest . U: . |
| Console output | 
https://builds.apache.org/job/PreCommit-HBASE-Build/8445/console |
| Powered by | Apache Yetus 0.4.0   http://yetus.apache.org |


This message was automatically generated.



> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-02 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.004 Put back the removal of MR dependency into hbase-rest; PE needs it.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723.master.004.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-02 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
18s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
21s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m 
40s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
49s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
46s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
19s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:red}-1{color} | {color:red} mvninstall {color} | {color:red}  0m 
18s{color} | {color:red} hbase-rest in the patch failed. {color} |
| {color:red}-1{color} | {color:red} mvninstall {color} | {color:red}  3m 
55s{color} | {color:red} root in the patch failed. {color} |
| {color:red}-1{color} | {color:red} compile {color} | {color:red}  0m 
21s{color} | {color:red} hbase-rest in the patch failed. {color} |
| {color:red}-1{color} | {color:red} compile {color} | {color:red}  2m  
7s{color} | {color:red} root in the patch failed. {color} |
| {color:red}-1{color} | {color:red} javac {color} | {color:red}  0m 21s{color} 
| {color:red} hbase-rest in the patch failed. {color} |
| {color:red}-1{color} | {color:red} javac {color} | {color:red}  2m  7s{color} 
| {color:red} root in the patch failed. {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
47s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m  
5s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  2m 
31s{color} | {color:red} The patch causes 24 errors with Hadoop v2.6.1. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  4m 
54s{color} | {color:red} The patch causes 24 errors with Hadoop v2.6.2. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  7m 
17s{color} | {color:red} The patch causes 24 errors with Hadoop v2.6.3. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red}  9m 
41s{color} | {color:red} The patch causes 24 errors with Hadoop v2.6.4. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red} 12m  
8s{color} | {color:red} The patch causes 24 errors with Hadoop v2.6.5. {color} |
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red} 14m 
30s{color} | {color:red} The patch causes 24 errors with Hadoop v2.7.1. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red} 16m 
56s{color} | {color:red} The patch causes 24 errors with Hadoop v2.7.2. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red} 19m 
19s{color} | {color:red} The patch causes 24 errors with Hadoop v2.7.3. {color} 
|
| {color:red}-1{color} | {color:red} hadoopcheck {color} | {color:red} 21m 
44s{color} | {color:red} The patch causes 24 errors with Hadoop v3.0.0-alpha4. 
{color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
46s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 10m 
25s{color} | {color:green} hbase-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  9m 
40s{color} | {color:green} hbase-backup in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  0m 20s{color} 
| {color:red} hbase-rest in the patch failed. {color} |
| 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-02 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.addendum More explicit includes of depdendencies (and removal of unused)  in 
hbase-backup, hbase-mapreduce, and in hbase-rest.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch, 
> HBASE-18723-pom-cleanup-Do-a-pass-with-dependency.addendum.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-02 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

Here is an addendum that comes of wrestling maven depdendency:analyze, hadoop 
exclusions and study of failures over on HBASE-18674. Lets see what this is 
like.

Cycle will be commit addendum, run HBASE-18674, find more holes, commit more 
here... but hopefully I got it this time.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


FAILURE: Integrated in Jenkins build HBase-2.0 #437 (See 
[https://builds.apache.org/job/HBase-2.0/437/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
43c4bc57618e3ede14f4964704ad753c27253e61)
* (edit) hbase-it/pom.xml
* (edit) hbase-resource-bundle/pom.xml
* (edit) hbase-thrift/pom.xml
* (edit) hbase-endpoint/pom.xml
* (edit) pom.xml
* (edit) hbase-hadoop-compat/pom.xml
* (edit) hbase-external-blockcache/pom.xml
* (edit) hbase-archetypes/hbase-client-project/pom.xml
* (edit) hbase-protocol-shaded/pom.xml
* (edit) hbase-examples/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) hbase-spark-it/pom.xml
* (edit) hbase-metrics/pom.xml
* (edit) hbase-server/pom.xml
* (edit) hbase-hadoop2-compat/pom.xml
* (edit) hbase-backup/pom.xml
* (edit) hbase-prefix-tree/pom.xml
* (edit) hbase-common/pom.xml
* (edit) hbase-rsgroup/pom.xml
* (edit) hbase-spark/pom.xml
* (edit) hbase-procedure/pom.xml
* (edit) hbase-shaded/pom.xml
* (edit) hbase-replication/pom.xml
* (edit) hbase-mapreduce/pom.xml
* (edit) hbase-shaded/hbase-shaded-check-invariants/pom.xml
* (edit) hbase-metrics-api/pom.xml
* (edit) hbase-shell/pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-01 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

Pushed to branch-2. Thanks [~Apache9]


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-01 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

bq. stack No branch-2 sir?

I forgot! Thanks for the kick.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-01 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18723:


SUCCESS: Integrated in Jenkins build HBase-Trunk_matrix #3636 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3636/])
HBASE-18723 [pom cleanup] Do a pass with dependency:analyze; remove (stack: rev 
fb537fe73695db4f648ec4f1fd54a99fa282bded)
* (edit) hbase-metrics/pom.xml
* (edit) hbase-shell/pom.xml
* (edit) hbase-external-blockcache/pom.xml
* (edit) hbase-common/pom.xml
* (edit) hbase-replication/pom.xml
* (edit) hbase-resource-bundle/pom.xml
* (edit) hbase-hadoop-compat/pom.xml
* (edit) hbase-shaded/pom.xml
* (edit) hbase-hadoop2-compat/pom.xml
* (edit) hbase-rsgroup/pom.xml
* (edit) pom.xml
* (edit) hbase-shaded/hbase-shaded-check-invariants/pom.xml
* (edit) hbase-thrift/pom.xml
* (edit) hbase-build-support/hbase-error-prone/pom.xml
* (edit) hbase-archetypes/hbase-client-project/pom.xml
* (edit) hbase-server/pom.xml
* (edit) hbase-spark/pom.xml
* (edit) hbase-procedure/pom.xml
* (edit) hbase-endpoint/pom.xml
* (edit) hbase-protocol-shaded/pom.xml
* (edit) hbase-backup/pom.xml
* (edit) hbase-it/pom.xml
* (edit) hbase-metrics-api/pom.xml
* (edit) hbase-examples/pom.xml
* (edit) hbase-spark-it/pom.xml
* (edit) hbase-prefix-tree/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) hbase-mapreduce/pom.xml


> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-09-01 Thread Duo Zhang (JIRA)

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

Duo Zhang commented on HBASE-18723:
---

[~stack] No branch-2 sir?

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
17s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
18s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
35s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m 
28s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  7m 
29s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  7m 
35s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
19s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 11m 
 0s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m 
28s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 10m 
28s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  7m 
26s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m 
33s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
33m  0s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  7m 
43s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
8s{color} | {color:green} hbase-error-prone in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
25s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
16s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
10s{color} | {color:green} hbase-metrics-api in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
21s{color} | {color:green} hbase-hadoop-compat in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-metrics in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
25s{color} | {color:green} hbase-hadoop2-compat in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
10s{color} | {color:green} hbase-replication in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
24s{color} | {color:green} hbase-prefix-tree in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
58s{color} | {color:green} hbase-procedure in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}108m  
9s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  9m 
31s{color} | {color:green} hbase-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
13s{color} | {color:green} hbase-resource-bundle in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
59s{color} | {color:green} 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.003 Put junit back in parent pom for now. Filed new issue to work on fixing 
our build so test is not run in all modules (lots of modules have no tests).

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch, HBASE-18723.master.003.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
25s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
16s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  9m 
53s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  7m 
 8s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  7m 
28s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
17s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  9m 
28s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  9m 
11s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  9m 
11s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  6m 
10s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m 
35s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
30m  6s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  7m  
2s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
8s{color} | {color:green} hbase-error-prone in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
25s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
16s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
8s{color} | {color:green} hbase-metrics-api in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
20s{color} | {color:green} hbase-hadoop-compat in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-metrics in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
26s{color} | {color:green} hbase-hadoop2-compat in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
9s{color} | {color:green} hbase-replication in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
26s{color} | {color:green} hbase-prefix-tree in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
54s{color} | {color:green} hbase-procedure in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 88m 
43s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  8m 
34s{color} | {color:green} hbase-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-resource-bundle in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
43s{color} | {color:green} 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread Umesh Agashe (JIRA)

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

Umesh Agashe commented on HBASE-18723:
--

+1, nice pom clean up [~stack]. I applied the patch locally, it builds. 
Compared the before and after output of command: "mvn dependency:analyze". 
Though there are a few warnings still present, patch cleaned up allot.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.002 Missed some junit and mockito includes.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch, 
> HBASE-18723.master.002.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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


[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-31 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18723:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
18s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} 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} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
45s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
52s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m 
28s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  7m 
30s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  7m 
35s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
19s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 10m 
55s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green} 10m 
27s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green} 10m 
27s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  7m 
23s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m 
33s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
33m 20s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  9m  
7s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
8s{color} | {color:green} hbase-error-prone in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  0m 29s{color} 
| {color:red} hbase-protocol-shaded in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
19s{color} | {color:green} hbase-common in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
10s{color} | {color:green} hbase-metrics-api in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
22s{color} | {color:green} hbase-hadoop-compat in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-metrics in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
26s{color} | {color:green} hbase-hadoop2-compat in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  0m 10s{color} 
| {color:red} hbase-replication in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
25s{color} | {color:green} hbase-prefix-tree in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  3m  
0s{color} | {color:green} hbase-procedure in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green} 88m 
15s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  8m  
1s{color} | {color:green} hbase-mapreduce in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
13s{color} | {color:green} hbase-resource-bundle in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
48s{color} | {color:green} hbase-thrift in the 

[jira] [Commented] (HBASE-18723) [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity list the dependencies we exploit

2017-08-30 Thread stack (JIRA)

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

stack commented on HBASE-18723:
---

.001

Do a pass with dependency:analyze; remove unused and
explicity list the dependencies we exploit

Remove the parent dependencies set which had junit, mockito,
log4j, and findbugs annotations. Listing in parent set meant
these libs were dependencies for all modules which in
practice was not the case. Edited all modules so those that
need any from this parent set now do explicit listing.

Ran the dependency:analyze over the project. Acted on most
suggested removals and requests for explicit listing. Some
grey areas remain around transitives that come in with
hadoop --needs better excludes, another project-- and that
the dependency:analyze tool is not always accurate in its
reporting.

> [pom cleanup] Do a pass with dependency:analyze; remove unused and explicity 
> list the dependencies we exploit
> -
>
> Key: HBASE-18723
> URL: https://issues.apache.org/jira/browse/HBASE-18723
> Project: HBase
>  Issue Type: Bug
>  Components: pom
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18723.master.001.patch
>
>
> Do a pass over our poms. They are sloppy including unused jars and not 
> listing actually used dependencies. Undo 'required' dependencies like junit 
> and mockito; not all modules need these anymore.
> This cleanup motivated by failures up on jenkins where a build step is not 
> finding transitive includes; explicit mention is needed (See failures in 
> HBASE-18674).



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