Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.20.0
    • Fix Version/s: 0.20.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Incompatible change
    • Release Note:
      Disabled Chukwa unit tests for 0.20 branch only.

      Description

      Chukwa is now a Hadoop sub project. The Chukwa code in the 0.20 branch is more than 3 months old and hasn't received many critical bug fixes. One problem we're running into is that one or more Chukwa unit tests frequently fail on 0.20 branch. This makes builds and releases of the branch more difficult.

      I propose we disable the Chukwa unit tests on 0.20 branch before we release Hadoop 0.20.0.

      Thoughts?

      1. HADOOP-5531.patch
        2.10 MB
        Nigel Daley

        Activity

        Owen O'Malley made changes -
        Component/s contrib/chukwa [ 12312445 ]
        Nigel Daley made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Nigel Daley made changes -
        Summary Disable Chukwa unit tests on branch-0.20 Remove Chukwa on branch-0.20
        Hide
        Nigel Daley added a comment -

        Update summary to reflect action taken.

        Show
        Nigel Daley added a comment - Update summary to reflect action taken.
        Robert Chansler made changes -
        Component/s contrib/chukwa [ 12312445 ]
        Robert Chansler made changes -
        Release Note Disabled Chukwa unit tests for 0.20 branch only.
        Nigel Daley made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Hadoop Flags [Incompatible change]
        Resolution Fixed [ 1 ]
        Hide
        Nigel Daley added a comment -

        I just committed this.

        Show
        Nigel Daley added a comment - I just committed this.
        Hide
        Nigel Daley added a comment -

        This patch is only for branch-0.20 which is why it failed Hudson.

        Show
        Nigel Daley added a comment - This patch is only for branch-0.20 which is why it failed Hudson.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12403461/HADOOP-5531.patch
        against trunk revision 757625.

        -1 @author. The patch appears to contain 2 @author tags which the Hadoop community has agreed to not allow in code contributions.

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

        -1 patch. The patch command could not apply the patch.

        Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/123/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/12403461/HADOOP-5531.patch against trunk revision 757625. -1 @author. The patch appears to contain 2 @author tags which the Hadoop community has agreed to not allow in code contributions. +1 tests included. The patch appears to include 103 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/123/console This message is automatically generated.
        Nigel Daley made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Assignee Nigel Daley [ nidaley ]
        Nigel Daley made changes -
        Field Original Value New Value
        Attachment HADOOP-5531.patch [ 12403461 ]
        Hide
        Nigel Daley added a comment -

        patch to remove chukwa from branch-0.20

        Show
        Nigel Daley added a comment - patch to remove chukwa from branch-0.20
        Hide
        Eric Yang added a comment -

        +1 for remove Chukwa from 0.20 branch.

        Show
        Eric Yang added a comment - +1 for remove Chukwa from 0.20 branch.
        Hide
        Jerome Boulon added a comment -

        +1 on removing Chukwa from branch-0.20

        Show
        Jerome Boulon added a comment - +1 on removing Chukwa from branch-0.20
        Hide
        Nigel Daley added a comment -

        I'm fine with Owen's suggestion of removing Chukwa from branch-0.20. Others?

        Show
        Nigel Daley added a comment - I'm fine with Owen's suggestion of removing Chukwa from branch-0.20. Others?
        Hide
        Owen O'Malley added a comment -

        I think we should either commit to fixing the bugs that are breaking the unit tests in 0.20 or remove Chukwa entirely from 0.20. I think disabling the unit tests, but leaving a broken Chukwa in the release is not very useful.

        Show
        Owen O'Malley added a comment - I think we should either commit to fixing the bugs that are breaking the unit tests in 0.20 or remove Chukwa entirely from 0.20. I think disabling the unit tests, but leaving a broken Chukwa in the release is not very useful.
        Hide
        Arun C Murthy added a comment -

        +1

        Show
        Arun C Murthy added a comment - +1
        Hide
        Jerome Boulon added a comment - - edited

        yes, the Junit is failing and there's 3 majors bugs that have been fixed in hadoop trunk after 0.20 branch.

        Show
        Jerome Boulon added a comment - - edited yes, the Junit is failing and there's 3 majors bugs that have been fixed in hadoop trunk after 0.20 branch.
        Hide
        Jerome Boulon added a comment -

        +1 on at least disabling the Chukwa unit tests on the 0.20 branch
        or remove Chukwa from the contrib directory since people should now be redirected to Chukwa sub-project

        Show
        Jerome Boulon added a comment - +1 on at least disabling the Chukwa unit tests on the 0.20 branch or remove Chukwa from the contrib directory since people should now be redirected to Chukwa sub-project
        Hide
        Owen O'Malley added a comment -

        Are they failing?

        Show
        Owen O'Malley added a comment - Are they failing?
        Hide
        Mac Yang added a comment -

        +1 on disabling the Chukwa unit tests on the 0.20 branch

        Show
        Mac Yang added a comment - +1 on disabling the Chukwa unit tests on the 0.20 branch
        Nigel Daley created issue -

          People

          • Assignee:
            Nigel Daley
            Reporter:
            Nigel Daley
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development