Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.6.0
    • Component/s: monitor, test
    • Labels:

      Description

      There's an orphaned test that never gets built/run, probably from a bad merge

      server/src/test/java/org/apache/accumulo/server/monitor/servlets/trace/SummaryTest.java
      

      With the per-service refactoring, there's no pom in server/ to cause this to be built. If you run through compile-test you'll see tehre is no target/ directory within server.

        Activity

        Sean Busbey created issue -
        Christopher Tubbs made changes -
        Field Original Value New Value
        Assignee Christopher Tubbs [ ctubbsii ]
        Christopher Tubbs made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        ASF subversion and git services added a comment -

        Commit 89d64c5bd0588e9dadfc076783d55aa000a12194 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Christopher Tubbs
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=89d64c5 ]

        ACCUMULO-2723 Put test in correct location

        Introduced by ACCUMULO-2631 bad merge

        Show
        ASF subversion and git services added a comment - Commit 89d64c5bd0588e9dadfc076783d55aa000a12194 in accumulo's branch refs/heads/1.6.0-SNAPSHOT from Christopher Tubbs [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=89d64c5 ] ACCUMULO-2723 Put test in correct location Introduced by ACCUMULO-2631 bad merge
        Hide
        ASF subversion and git services added a comment -

        Commit 89d64c5bd0588e9dadfc076783d55aa000a12194 in accumulo's branch refs/heads/master from Christopher Tubbs
        [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=89d64c5 ]

        ACCUMULO-2723 Put test in correct location

        Introduced by ACCUMULO-2631 bad merge

        Show
        ASF subversion and git services added a comment - Commit 89d64c5bd0588e9dadfc076783d55aa000a12194 in accumulo's branch refs/heads/master from Christopher Tubbs [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=89d64c5 ] ACCUMULO-2723 Put test in correct location Introduced by ACCUMULO-2631 bad merge
        Hide
        Sean Busbey added a comment -

        Is it worth a follow on ticket to have the enforcer plugin check that source paths likely to indicate bad merges are empty?

        Show
        Sean Busbey added a comment - Is it worth a follow on ticket to have the enforcer plugin check that source paths likely to indicate bad merges are empty?
        Christopher Tubbs made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.6.0 [ 12322468 ]
        Fix Version/s 1.7.0 [ 12324607 ]
        Fix Version/s 1.6.1 [ 12325441 ]
        Resolution Fixed [ 1 ]
        Christopher Tubbs made changes -
        Affects Version/s 1.6.0 [ 12322468 ]
        Hide
        Christopher Tubbs added a comment -

        I didn't know the enforcer plugin could do that, but perhaps it'd be useful.

        Show
        Christopher Tubbs added a comment - I didn't know the enforcer plugin could do that, but perhaps it'd be useful.
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        8h 10m 1 Christopher Tubbs 23/Apr/14 14:57
        In Progress In Progress Resolved Resolved
        14m 23s 1 Christopher Tubbs 23/Apr/14 15:12

          People

          • Assignee:
            Christopher Tubbs
            Reporter:
            Sean Busbey
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development