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

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

          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.
          Show
          Timothy St. Clair added a comment - ref: http://koji.fedoraproject.org/koji/packageinfo?packageID=9762
          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
          Hide
          Tsz Wo Nicholas Sze added a comment -

          Added Tim.

          Show
          Tsz Wo Nicholas Sze added a comment - Added Tim.
          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
          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-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
          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-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

            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