Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.23.0
    • Component/s: None
    • Labels:
      None

      Description

      Currently yarn.version and hadoop-mapreduce.version are set to be 1.0, clearly it's 0.23.0.

      Also, we should stop using $

      {yarn.version}

      and $

      {hadoop-mapreduce.version}

      in all the poms, maven doesn't like the version substitutions - it complains bitterly!

      1. version.patch
        21 kB
        Arun C Murthy
      2. version.patch
        7 kB
        Arun C Murthy

        Issue Links

          Activity

          Arun C Murthy created issue -
          Hide
          Arun C Murthy added a comment -

          Straight-fwd patch, I've fixed INSTALL too.

          Show
          Arun C Murthy added a comment - Straight-fwd patch, I've fixed INSTALL too.
          Arun C Murthy made changes -
          Field Original Value New Value
          Attachment version.patch [ 12491673 ]
          Arun C Murthy made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Alejandro Abdelnur added a comment -

          FYI, not only that maven complains, but parameterized project <version> elements getting the property from the parent are unusable when deployed in a Maven repo.

          Show
          Alejandro Abdelnur added a comment - FYI, not only that maven complains, but parameterized project <version> elements getting the property from the parent are unusable when deployed in a Maven repo.
          Hide
          Arun C Murthy added a comment -

          Alejandro - is the updated patch reasonable? Thanks.

          Show
          Arun C Murthy added a comment - Alejandro - is the updated patch reasonable? Thanks.
          Hide
          Luke Lu added a comment -

          Actually the maven complaints in this case is bogus as the actual deployed pom is target/saner-pom.xml with version substituted. The current POMs are very CI friendly (see discussion in MNG-2971). The patch makes CI almost impossible. Make sure Giri take a look at this

          Also, even if we want to get rid of version properties to appease the bogus complaints, this patch is not OK, as it's not maven versions plugin friendly.

          Show
          Luke Lu added a comment - Actually the maven complaints in this case is bogus as the actual deployed pom is target/saner-pom.xml with version substituted. The current POMs are very CI friendly (see discussion in MNG-2971 ). The patch makes CI almost impossible. Make sure Giri take a look at this Also, even if we want to get rid of version properties to appease the bogus complaints, this patch is not OK, as it's not maven versions plugin friendly.
          Arun C Murthy made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Arun C Murthy added a comment -

          Simpler patch based on Luke's comments.

          Show
          Arun C Murthy added a comment - Simpler patch based on Luke's comments.
          Arun C Murthy made changes -
          Attachment version.patch [ 12491677 ]
          Arun C Murthy made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Luke Lu added a comment -

          +1 on the v2 patch. Opening a jira for MRv2 classpath management

          Show
          Luke Lu added a comment - +1 on the v2 patch. Opening a jira for MRv2 classpath management
          Hide
          Arun C Murthy added a comment -

          Thanks for the review Luke.

          I've just committed this.

          Show
          Arun C Murthy added a comment - Thanks for the review Luke. I've just committed this.
          Arun C Murthy made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Luke Lu made changes -
          Link This issue relates to MAPREDUCE-2880 [ MAPREDUCE-2880 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #785 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/785/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0.

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #785 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/785/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0. acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705 Files : /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #788 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/788/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0.

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #788 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/788/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0. acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705 Files : /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #862 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/862/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0.

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #862 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/862/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0. acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705 Files : /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #792 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/792/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #792 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/792/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0 acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722 Files : /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #788 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/788/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #788 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/788/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0 acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722 Files : /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk-Commit #865 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/865/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk-Commit #865 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/865/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0 acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722 Files : /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Hdfs-trunk #768 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/768/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0.

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Hdfs-trunk #768 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/768/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0 MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0. acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722 Files : /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705 Files : /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #793 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/793/)
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0
          MAPREDUCE-2879. Fix version for MR-279 to 0.23.0.

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties

          acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705
          Files :

          • /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt
          • /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml
          • /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #793 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/793/ ) MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0 MAPREDUCE-2879 . Fix version for MR-279 to 0.23.0. acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161722 Files : /hadoop/common/trunk/hadoop-mapreduce-project/ivy/libraries.properties acmurthy : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1161705 Files : /hadoop/common/trunk/hadoop-mapreduce-project/CHANGES.txt /hadoop/common/trunk/hadoop-mapreduce-project/INSTALL /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-common/src/main/java/org/apache/hadoop/mapreduce/v2/MRConstants.java /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/resources/yarn-default.xml /hadoop/common/trunk/hadoop-mapreduce-project/hadoop-yarn/pom.xml /hadoop/common/trunk/hadoop-mapreduce-project/pom.xml
          Eli Collins made changes -
          Fix Version/s 0.23.0 [ 12315570 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Patch Available Patch Available Open Open
          23m 56s 1 Arun C Murthy 25/Aug/11 19:58
          Open Open Patch Available Patch Available
          4m 52s 2 Arun C Murthy 25/Aug/11 20:01
          Patch Available Patch Available Resolved Resolved
          13m 10s 1 Arun C Murthy 25/Aug/11 20:14

            People

            • Assignee:
              Arun C Murthy
              Reporter:
              Arun C Murthy
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development