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