Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.23.0
    • Fix Version/s: 0.23.0
    • Component/s: build
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Currently the build creates 2 different tarball layouts.

      One is the legacy one, the layout used until 0.22 (ant tar & mvn package -Ptar)

      The other is new new one, the layout used in trunk that mimics the Unix layout (ant binary & mvn package -Pbintar).

      The legacy layout is of not use as all the scripts have been modified to work with the new layout only.

      We should thus remove the legacy layout generation.

      In addition we could rename the current 'bintar' to just 'tar'

      1. HADOOP-7498v1.patch
        15 kB
        Alejandro Abdelnur

        Issue Links

          Activity

          Hide
          Tom White added a comment -

          I've just committed this.

          Show
          Tom White added a comment - I've just committed this.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #762 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/762/)
          HADOOP-7498. Remove legacy TAR layout creation. Contributed by Alejandro Abdelnur.

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

          • /hadoop/common/trunk/hadoop-assemblies/src/main/resources/assemblies/hadoop-bintar.xml
          • /hadoop/common/trunk/hadoop-assemblies/src/main/resources/assemblies/hadoop-tar.xml
          • /hadoop/common/trunk/hadoop-project-distro/pom.xml
          • /hadoop/common/trunk/hadoop-common/CHANGES.txt
          • /hadoop/common/trunk/BUILDING.txt
          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #762 (See https://builds.apache.org/job/Hadoop-Common-trunk-Commit/762/ ) HADOOP-7498 . Remove legacy TAR layout creation. Contributed by Alejandro Abdelnur. tomwhite : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1160341 Files : /hadoop/common/trunk/hadoop-assemblies/src/main/resources/assemblies/hadoop-bintar.xml /hadoop/common/trunk/hadoop-assemblies/src/main/resources/assemblies/hadoop-tar.xml /hadoop/common/trunk/hadoop-project-distro/pom.xml /hadoop/common/trunk/hadoop-common/CHANGES.txt /hadoop/common/trunk/BUILDING.txt
          Hide
          Eric Yang added a comment -

          +1 looks good.

          Show
          Eric Yang added a comment - +1 looks good.
          Hide
          Alejandro Abdelnur added a comment -

          Current patch works on top of HDFS-2096 version 'AJ'

          Show
          Alejandro Abdelnur added a comment - Current patch works on top of HDFS-2096 version 'AJ'
          Hide
          Alejandro Abdelnur added a comment -

          It works both for common & hdfs

          Show
          Alejandro Abdelnur added a comment - It works both for common & hdfs
          Hide
          Alejandro Abdelnur added a comment -

          Removes legacy tar creation.

          Renames remaining profile to be 'tar'

          The TAR itself does not have the '-bin' postfix anymore.

          Show
          Alejandro Abdelnur added a comment - Removes legacy tar creation. Renames remaining profile to be 'tar' The TAR itself does not have the '-bin' postfix anymore.
          Hide
          Eric Yang added a comment -

          +1 for removing the legacy tar layout. With result of the discussion on HADOOP-7521, we should support using both isolated directory layout as well as merged layout. Hence, the shell script needs to check and act correctly to the corresponding layout.

          Show
          Eric Yang added a comment - +1 for removing the legacy tar layout. With result of the discussion on HADOOP-7521 , we should support using both isolated directory layout as well as merged layout. Hence, the shell script needs to check and act correctly to the corresponding layout.
          Hide
          Giridharan Kesavan added a comment -

          +1 for removing the legacy tar layout and renaming bintar to tar.

          Show
          Giridharan Kesavan added a comment - +1 for removing the legacy tar layout and renaming bintar to tar.

            People

            • Assignee:
              Alejandro Abdelnur
              Reporter:
              Alejandro Abdelnur
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development