ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-534

The test target in contib/bookkeeper does not depend on jar target.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.0, 3.2.1
    • Fix Version/s: 3.3.0
    • Component/s: contrib-bookkeeper
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      The test target in contib/bookkeeper does not depend on jar target. So the ant test target gives compilation errors if the main is not compiled which can be prevented if it depends on jar. It can then check if main has been compiled or not and throw out a reasonable error.

        Issue Links

          Activity

          Mahadev konar created issue -
          Mahadev konar made changes -
          Field Original Value New Value
          Fix Version/s 3.3.0 [ 12313976 ]
          Affects Version/s 3.2.1 [ 12314068 ]
          Affects Version/s 3.2.0 [ 12313491 ]
          Component/s contrib-bookkeeper [ 12312643 ]
          Patrick Hunt made changes -
          Assignee Patrick Hunt [ phunt ]
          Hide
          Patrick Hunt added a comment -

          updated bk and rest contribs to get "common" (ie log4j and junit) jars using ivy

          fixed bk init to depend on checkmain, now always checks for zk being compiled first and gives sane warning

          Show
          Patrick Hunt added a comment - updated bk and rest contribs to get "common" (ie log4j and junit) jars using ivy fixed bk init to depend on checkmain, now always checks for zk being compiled first and gives sane warning
          Patrick Hunt made changes -
          Attachment ZOOKEEPER-534.patch [ 12426643 ]
          Patrick Hunt made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Mahadev konar added a comment -

          trying hudson again.

          Show
          Mahadev konar added a comment - trying hudson again.
          Mahadev konar made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Mahadev konar made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Mahadev konar added a comment -

          +1 the patch looks good. ill commit as soon as hudson ok's it.

          Show
          Mahadev konar added a comment - +1 the patch looks good. ill commit as soon as hudson ok's it.
          Hide
          Mahadev konar added a comment -

          +1 the patch looks good. ill commit as soon as hudson ok's it.

          Show
          Mahadev konar added a comment - +1 the patch looks good. ill commit as soon as hudson ok's it.
          Hide
          Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12426643/ZOOKEEPER-534.patch
          against trunk revision 889848.

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 5 new or modified tests.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed core unit tests.

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - +1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12426643/ZOOKEEPER-534.patch against trunk revision 889848. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 5 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: http://hudson.zones.apache.org/hudson/job/Zookeeper-Patch-h7.grid.sp2.yahoo.net/25/console This message is automatically generated.
          Flavio Junqueira made changes -
          Link This issue depends on ZOOKEEPER-623 [ ZOOKEEPER-623 ]
          Hide
          Flavio Junqueira added a comment -

          +1 for this patch. However, to have compilation working 100%, we need ZOOKEEPER-623 as well.

          Show
          Flavio Junqueira added a comment - +1 for this patch. However, to have compilation working 100%, we need ZOOKEEPER-623 as well.
          Hide
          Mahadev konar added a comment -

          I just committed this. thanks pat.

          Show
          Mahadev konar added a comment - I just committed this. thanks pat.
          Mahadev konar made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags [Reviewed]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in ZooKeeper-trunk #632 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/632/)

          Show
          Hudson added a comment - Integrated in ZooKeeper-trunk #632 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/632/ )
          Patrick Hunt made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Link This issue depends on ZOOKEEPER-623 [ ZOOKEEPER-623 ]
          Gavin made changes -
          Link This issue depends upon ZOOKEEPER-623 [ ZOOKEEPER-623 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Patch Available Patch Available Open Open
          9d 14h 26m 1 Mahadev konar 11/Dec/09 23:12
          Open Open Patch Available Patch Available
          69d 13h 59m 2 Mahadev konar 11/Dec/09 23:13
          Patch Available Patch Available Resolved Resolved
          5d 33m 1 Mahadev konar 16/Dec/09 23:46
          Resolved Resolved Closed Closed
          99d 17h 38m 1 Patrick Hunt 26/Mar/10 17:25

            People

            • Assignee:
              Patrick Hunt
              Reporter:
              Mahadev konar
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development