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

        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?
        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.

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development