Details

    • Sub-task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • None
    • mrv2
    • None

    Description

      Fixes two bugs in DataStatistics: a divide by zero in the variance calculation when count == 0, and a synchronization issue in how the updateStatistics method was implemented.

      Attachments

        1. MR-2642-1.patch
          5 kB
          Josh Wills
        2. MAPREDUCE-2642.patch
          4 kB
          Josh Wills

        Activity

          hadoopqa Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12485293/MAPREDUCE-2642.patch
          against trunk revision 1145424.

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

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

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

          Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/455//console

          This message is automatically generated.

          hadoopqa Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12485293/MAPREDUCE-2642.patch against trunk revision 1145424. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified tests. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/455//console This message is automatically generated.
          acmurthy Arun Murthy added a comment -

          +1

          Josh, can you pls re-gen the patch. It's a little stale.

          acmurthy Arun Murthy added a comment - +1 Josh, can you pls re-gen the patch. It's a little stale.
          jwills Josh Wills added a comment -

          Apologies Arun-- re-gen'd patch attached.

          jwills Josh Wills added a comment - Apologies Arun-- re-gen'd patch attached.
          acmurthy Arun Murthy added a comment -

          I just committed this. Thanks Josh!

          acmurthy Arun Murthy added a comment - I just committed this. Thanks Josh!

          People

            jwills Josh Wills
            jwills Josh Wills
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: