Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0, 2.1.0-beta
    • Fix Version/s: 2.3.0
    • Component/s: build
    • Labels:

      Description

      Current dependency is against 2.1 which has been deprecated, and numerous bugz have been fixed since then. See http://commons.apache.org/proper/commons-math/changes-report.html for details. This also affects mvn-rpmbuild when running on newer platforms e.g. Fedora 18.

      1. HADOOP-9594.patch
        2 kB
        Timothy St. Clair

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          22m 20s 1 Timothy St. Clair 23/May/13 21:55
          Patch Available Patch Available Resolved Resolved
          173d 22h 48m 1 Steve Loughran 13/Nov/13 19:43
          Resolved Resolved Closed Closed
          103d 1h 14m 1 Arun C Murthy 24/Feb/14 20:58
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Arun C Murthy made changes -
          Fix Version/s 2.3.0 [ 12325254 ]
          Fix Version/s 2.4.0 [ 12324587 ]
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Hdfs-trunk #1582 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1582/)
          HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672)

          • /hadoop/common/trunk
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Hdfs-trunk #1582 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/1582/ ) HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672 ) /hadoop/common/trunk /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java /hadoop/common/trunk/hadoop-project/pom.xml
          Hide
          Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk #1608 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1608/)
          HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672)

          • /hadoop/common/trunk
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          Show
          Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #1608 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1608/ ) HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672 ) /hadoop/common/trunk /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java /hadoop/common/trunk/hadoop-project/pom.xml
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Yarn-trunk #391 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/391/)
          HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672)

          • /hadoop/common/trunk
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-Yarn-trunk #391 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/391/ ) HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672 ) /hadoop/common/trunk /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java /hadoop/common/trunk/hadoop-project/pom.xml
          Hide
          Steve Loughran added a comment -

          oh, and +1 (& thanks Nicholas)

          Show
          Steve Loughran added a comment - oh, and +1 (& thanks Nicholas)
          Hide
          Hudson added a comment -

          SUCCESS: Integrated in Hadoop-trunk-Commit #4727 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4727/)
          HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672)

          • /hadoop/common/trunk
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java
          • /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java
          • /hadoop/common/trunk/hadoop-project/pom.xml
          Show
          Hudson added a comment - SUCCESS: Integrated in Hadoop-trunk-Commit #4727 (See https://builds.apache.org/job/Hadoop-trunk-Commit/4727/ ) HADOOP-9594 commons-math (stevel: http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1541672 ) /hadoop/common/trunk /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt /hadoop/common/trunk/hadoop-common-project/hadoop-common/pom.xml /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/impl/MetricsSystemImpl.java /hadoop/common/trunk/hadoop-project/pom.xml
          Tsz Wo Nicholas Sze made changes -
          Assignee Timothy St. Clair [ tstclair ]
          Hide
          Tsz Wo Nicholas Sze added a comment -

          Added Tim.

          Show
          Tsz Wo Nicholas Sze added a comment - Added Tim.
          Steve Loughran made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Steve Loughran added a comment -

          Fixed in 2.3.0 -I'd assign it to tstclair in the assignee field except JIRA isn't letting me add new contributors right now. Tim: you get your credit in CHANGES.TXT for now

          Show
          Steve Loughran added a comment - Fixed in 2.3.0 -I'd assign it to tstclair in the assignee field except JIRA isn't letting me add new contributors right now. Tim: you get your credit in CHANGES.TXT for now
          Steve Loughran made changes -
          Link This issue is depended upon by HADOOP-9991 [ HADOOP-9991 ]
          Arun C Murthy made changes -
          Fix Version/s 2.3.0 [ 12324587 ]
          Fix Version/s 2.1.0-beta [ 12324030 ]
          Show
          Timothy St. Clair added a comment - ref: http://koji.fedoraproject.org/koji/packageinfo?packageID=9762
          Hide
          Timothy St. Clair added a comment -

          2.X line is deprecated (last rev was 2011-03-02) and is no longer supported in major distros. This patch is meant to address that drift (mvn-rpmbuild), not to mention that there are numerous fixes and optimizations since then.

          The patch also addresses the api change.

          Show
          Timothy St. Clair added a comment - 2.X line is deprecated (last rev was 2011-03-02) and is no longer supported in major distros. This patch is meant to address that drift (mvn-rpmbuild), not to mention that there are numerous fixes and optimizations since then. The patch also addresses the api change.
          Hide
          Luke Lu added a comment -

          This is an incompatible change. Can you tell me why can't we just upgrade to 2.2? BTW, math 3.x and 2.x can be used side by side as the package name for 3.x is in math3.

          Show
          Luke Lu added a comment - This is an incompatible change. Can you tell me why can't we just upgrade to 2.2? BTW, math 3.x and 2.x can be used side by side as the package name for 3.x is in math3.
          Hide
          Timothy St. Clair added a comment -

          This ticket has gotten little attention and is pretty trivial any watchers want to review?

          Show
          Timothy St. Clair added a comment - This ticket has gotten little attention and is pretty trivial any watchers want to review?
          Timothy St. Clair made changes -
          Description Current dependency is against 2.1 which has been deprecated, and numerous bugz have been fixed since then. See http://commons.apache.org/proper/commons-math/changes-report.html for details. Current dependency is against 2.1 which has been deprecated, and numerous bugz have been fixed since then. See http://commons.apache.org/proper/commons-math/changes-report.html for details. This also affects mvn-rpmbuild when running on newer platforms e.g. Fedora 18.
          Timothy St. Clair made changes -
          Labels newbie maven newbie
          Hide
          Timothy St. Clair added a comment -

          Ready for review imho, as it is primarily a simple POM modification.

          Show
          Timothy St. Clair added a comment - Ready for review imho, as it is primarily a simple POM modification.
          Hide
          Hadoop QA added a comment -

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

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

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

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

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

          +1 eclipse:eclipse. The patch built with eclipse:eclipse.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

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

          +1 core tests. The patch passed unit tests in hadoop-common-project/hadoop-common.

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

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2562//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2562//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/12584578/HADOOP-9594.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . The javadoc tool did not generate any warning messages. +1 eclipse:eclipse . The patch built with eclipse:eclipse. +1 findbugs . The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in hadoop-common-project/hadoop-common. +1 contrib tests . The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/2562//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/2562//console This message is automatically generated.
          Timothy St. Clair made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Fix Version/s 2.0.5-beta [ 12324030 ]
          Timothy St. Clair made changes -
          Field Original Value New Value
          Attachment HADOOP-9594.patch [ 12584578 ]
          Hide
          Timothy St. Clair added a comment -

          Simple modification to support math commons 3.1.1

          Show
          Timothy St. Clair added a comment - Simple modification to support math commons 3.1.1
          Timothy St. Clair created issue -

            People

            • Assignee:
              Timothy St. Clair
              Reporter:
              Timothy St. Clair
            • Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development