[ 
https://issues.apache.org/jira/browse/YETUS-280?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Sean Busbey updated YETUS-280:
------------------------------
    Attachment: HBASE-15075.yetus.log

I'm attaching the log from running master + this patch against Ted's issue 
HBASE-15075

{code}
-1 overall

 _____     _ _                _ 
|  ___|_ _(_) |_   _ _ __ ___| |
| |_ / _` | | | | | | '__/ _ \ |
|  _| (_| | | | |_| | | |  __/_|
|_|  \__,_|_|_|\__,_|_|  \___(_)
                                


| Vote |       Subsystem |  Runtime   | Comment
============================================================================
|  +1  |      hbaseanti  |  0m 00s    | Patch does not have any anti-patterns. 
|  +1  |        @author  |  0m 00s    | The patch does not contain any @author 
|      |                 |            | tags.
|  +1  |     test4tests  |  0m 00s    | The patch appears to include 6 new or 
|      |                 |            | modified test files.
|   0  |         mvndep  |  0m 28s    | Maven dependency ordering for branch 
|  +1  |     mvninstall  |  6m 39s    | master passed 
|  +1  |        compile  |  3m 25s    | master passed 
|  +1  |     checkstyle  |  9m 27s    | master passed 
|  +1  |     mvneclipse  |  1m 00s    | master passed 
|  +1  |    hbaseprotoc  |  1m 17s    | the patch passed 
|   0  |       findbugs  |  1m 28s    | hbase-client in master has 13 extant 
|      |                 |            | Findbugs warnings.
|   0  |       findbugs  |  2m 44s    | hbase-server in master has 83 extant 
|      |                 |            | Findbugs warnings.
|  +1  |        javadoc  |  1m 24s    | master passed 
|   0  |         mvndep  |  0m 30s    | Maven dependency ordering for patch 
|  +1  |     mvninstall  |  2m 53s    | the patch passed 
|  +1  |        compile  |  3m 23s    | the patch passed 
|  +1  |             cc  |  3m 23s    | the patch passed 
|  -1  |          javac  |  6m 46s    | hbase-server generated 2 new issues 
|      |                 |            | (was 6, now 6).
|  +1  |          javac  |  3m 23s    | the patch passed 
|  -1  |     checkstyle  |  3m 25s    | Patch generated 32 new checkstyle 
|      |                 |            | issues in hbase-server (total was 517,
|      |                 |            | now 529).
|  +1  |     mvneclipse  |  0m 58s    | the patch passed 
|  -1  |     whitespace  |  0m 00s    | The patch has 4 line(s) that end in 
|      |                 |            | whitespace. Use git apply
|      |                 |            | --whitespace=fix.
|  +1  |    hbaseprotoc  |  1m 15s    | the patch passed 
|  +1  |       findbugs  |  6m 52s    | the patch passed 
|  +1  |        javadoc  |  1m 49s    | the patch passed 
|  +1  |     asflicense  |  0m 50s    | Patch does not generate ASF License 
|      |                 |            | warnings.
|      |                 |  58m 32s   | 
{code}

Looks like the new change properly placed things in dependency order.

+1.

> build systems should be able to predetermine module order
> ---------------------------------------------------------
>
>                 Key: YETUS-280
>                 URL: https://issues.apache.org/jira/browse/YETUS-280
>             Project: Yetus
>          Issue Type: Improvement
>          Components: Test Patch
>    Affects Versions: 0.1.0
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>         Attachments: HBASE-15075.yetus.log, YETUS-280.00.patch, 
> YETUS-280.01.patch
>
>
> It'd be useful for systems that have enough metadata to provide Yetus with 
> hints on the order of the modules.



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

Reply via email to