[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

They are in place thoughout, yes. They were added once and then have not been 
maintained since. Original idea was that we could feed hbase piecemeal to 
external CI sites like Travis -- Travis doesn't like projects of our size and 
test-duration -- first passing it the i/o tests, then the region tests, then 
the mapreduce tests, and so on. It cross-cuts our modularization. Not sure it 
the way to go. Either we double-down on the "test-type" categorization -- we 
already have another categorization dimension of small/medium/large which has 
decent definition up in refguide -- and exploit it in someway (at mo. it is 
just unused noise) or we purge it as a failed experiment. I'm for the latter.

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-18782:
-

I see the test categorizations, which I thought we used extensively?

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

bq. Is that fine

Yes.

After yetus goes in, whats in hbase-annotation is nought but categorizations, 
categorizations we do not make use of that I know of; either we double-down on 
the categorizations or remove them (and hbase-annotations module).

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread Sean Busbey (JIRA)

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

Sean Busbey commented on HBASE-18782:
-

I'm rebasing my work to move to yetus audience annotation in HBASE-17823 and I 
just got to the impact of this commit. to make sure I'm not undoing any of your 
intent here [~stack], I wanted to check in. At the moment, I've got the 
dependency on yetus annotations put into the hbase-build-configuration pom, 
essentially using it as short hand for "any hbase module with java code should 
have this dependency". With that change in place, the few modules where you've 
removed annotation (e.g. hbase-external-blockcache) will have the annotations 
jar as a dependency and I can put back in the explicit IA.Private. Is that 
fine, or would you prefer I make it so that the dep doesn't show up?

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18782:


FAILURE: Integrated in Jenkins build HBase-2.0 #493 (See 
[https://builds.apache.org/job/HBase-2.0/493/])
HBASE-18782 Module untangling work (stack: rev 
7be38cbc6a96fe0d9fa9ce49379982ebdcfbbad8)
* (edit) hbase-archetypes/hbase-client-project/pom.xml
* (edit) 
hbase-protocol/src/main/java/org/apache/hadoop/hbase/util/ByteStringer.java
* (edit) hbase-examples/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) hbase-external-blockcache/pom.xml
* (edit) 
hbase-protocol-shaded/src/main/java/org/apache/hadoop/hbase/util/ForeignExceptionUtil.java
* (edit) hbase-it/pom.xml
* (edit) hbase-protocol/pom.xml
* (edit) hbase-protocol-shaded/pom.xml
* (edit) hbase-spark-it/pom.xml
* (edit) 
hbase-external-blockcache/src/main/java/org/apache/hadoop/hbase/io/hfile/MemcachedBlockCache.java


> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread Hudson (JIRA)

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

Hudson commented on HBASE-18782:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #3695 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/3695/])
HBASE-18782 Module untangling work (stack: rev 
591d86ab29bfebe1e87657c030319d17c7c8ef44)
* (edit) hbase-external-blockcache/pom.xml
* (edit) 
hbase-protocol/src/main/java/org/apache/hadoop/hbase/util/ByteStringer.java
* (edit) 
hbase-protocol-shaded/src/main/java/org/apache/hadoop/hbase/util/ForeignExceptionUtil.java
* (edit) hbase-examples/pom.xml
* (edit) hbase-rest/pom.xml
* (edit) hbase-spark-it/pom.xml
* (edit) hbase-it/pom.xml
* (edit) hbase-archetypes/hbase-client-project/pom.xml
* (edit) 
hbase-external-blockcache/src/main/java/org/apache/hadoop/hbase/io/hfile/MemcachedBlockCache.java
* (edit) hbase-protocol-shaded/pom.xml
* (edit) hbase-protocol/pom.xml


> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Fix For: 2.0.0-alpha-3
>
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

Oh, thanks for review [~appy]

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-10 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

bq. imo, 5 lines comments on top of classes don't look any better. 

They allow undo of the tangle brought on by one class only.

bq. But it's a nit right now, and meat of the change is untangling other deps.

Yeah.

The move to use yetus annotations makes it so we could near do away with 
hbase-annotations altogether. Remaining are typing labels we don't honor. We 
should either take them on or just strip them.



> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-09 Thread Appy (JIRA)

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

Appy commented on HBASE-18782:
--

lgtm.
imo, 5 lines comments on top of classes don't look any better. :-)
I think things will drift back to everything depending on annotations again, 
which kind of makes sense to me. I guess in any future, if we want hbase 
classes to be well labelled, that's the only way(?). Btw, I filtered out that 
module from my structure101 view.
But it's a nit right now, and meat of the change is untangling other deps.
So +1.


> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-09 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18782:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  1m 
27s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {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 
19s{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}  2m 
21s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
 6s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  1m 
51s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
47s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
28s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
20s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  2m 
35s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  2m 
21s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
 6s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  1m 
52s{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 
10s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
34m 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:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  5m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
31s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
27s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
20s{color} | {color:green} hbase-protocol in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
20s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
53s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
56s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
12s{color} | {color:green} hbase-external-blockcache in the patch passed. 
{color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-spark-it in the p

[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-09 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

.004 Don't change hbase-shell pom. Be conservative.

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch, HBASE-18782.master.002.patch, 
> HBASE-18782.master.003.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-09 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18782:
---

| (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} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {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 
19s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
46s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
36s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
21s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
15s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  5m 
13s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
47s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
20s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  2m 
46s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  2m 
41s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
16s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
 5s{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 
10s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
34m 21s{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:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  5m 
26s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
38s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
28s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
19s{color} | {color:green} hbase-protocol in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  6m 
50s{color} | {color:green} hbase-shell in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
20s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
48s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
45s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
12s{color} | {color:green} hbase-external-blockcache in the patch

[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18782:
---

| (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} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {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 
19s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
 1s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
45s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
15s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
 3s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  4m 
30s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
33s{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}  2m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
27s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  2m 
27s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
12s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  2m 
 3s{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 
10s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
35m 31s{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:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  6m 
13s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
36s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
27s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
19s{color} | {color:green} hbase-protocol in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  7m  3s{color} 
| {color:red} hbase-shell in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
23s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
56s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  3m  
3s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
13s{color} | {color:green} hbase-external-blockcache in the patch passed.

[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-08 Thread stack (JIRA)

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

stack commented on HBASE-18782:
---

.002 is less ambitious.

> Module untangling work
> --
>
> Key: HBASE-18782
> URL: https://issues.apache.org/jira/browse/HBASE-18782
> Project: HBase
>  Issue Type: Task
>Reporter: stack
>Assignee: stack
> Attachments: HBASE-18782.master.001.patch, 
> HBASE-18782.master.002.patch
>
>
> Here is a bit of module untangling work done using structure 101. In a few 
> places we depend on modules but are not making use of the dependency. In 
> other places we depended on a module just because we had one references only, 
> one we could undo.
> After this little exercise, we have nicer layering and stuff like 
> hbase-protocol and hbase-protocol-shaded no longer depend on other hbase 
> components.  Not everything depends on annotations anymore.
> There is way more work to do here but this will do for now.



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


[jira] [Commented] (HBASE-18782) Module untangling work

2017-09-08 Thread Hadoop QA (JIRA)

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

Hadoop QA commented on HBASE-18782:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  4m  
1s{color} | {color:blue} Docker mode activated. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {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 
 1s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m 
24s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
46s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  3m 
 0s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  6m 
58s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  2m 
33s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
22s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
31s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  4m 
17s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  4m 
17s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} mvneclipse {color} | {color:green}  3m 
 1s{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 
15s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
35m 50s{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:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Skipped patched modules with no Java source: 
hbase-spark-it {color} |
| {color:green}+1{color} | {color:green} findbugs {color} | {color:green}  7m 
15s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  2m  
4s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
28s{color} | {color:green} hbase-protocol-shaded in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
20s{color} | {color:green} hbase-protocol in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  7m  2s{color} 
| {color:red} hbase-shell in the patch failed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}  0m 58s{color} 
| {color:red} hbase-endpoint in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
19s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
48s{color} | {color:green} hbase-examples in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
42s{color} | {color:green} hbase-rest in the patch passed. {color} |
| {color