Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-13964

Remove vestigal templates directories creation

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0-alpha1
    • Fix Version/s: 3.0.0-alpha2
    • Component/s: build
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Incompatible change
    • Release Note:
      This patch removes share/hadoop/{hadoop,hdfs,mapred,yarn}/templates directories and contents.

      Description

      the share/hadoop/(component)/template directories are from when RPM and such were built as part of the build system. that no longer happens and now those files cause more harm than good since they are in the classpath. let's remove them.

      1. HADOOP-13964.00.patch
        4 kB
        Allen Wittenauer

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11137 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11137/)
        HADOOP-13964. Remove vestigal templates directories (aw: rev acda1a5fee5522a5c1e31587f4ccce5fdd1af74d)

        • (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-mapreduce-dist.xml
        • (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-yarn-dist.xml
        • (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-dist.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11137 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11137/ ) HADOOP-13964 . Remove vestigal templates directories (aw: rev acda1a5fee5522a5c1e31587f4ccce5fdd1af74d) (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-mapreduce-dist.xml (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-yarn-dist.xml (edit) hadoop-assemblies/src/main/resources/assemblies/hadoop-dist.xml
        Hide
        andrew.wang Andrew Wang added a comment -

        Show
        andrew.wang Andrew Wang added a comment -
        Hide
        aw Allen Wittenauer added a comment -

        Ha. I beat it.

        Show
        aw Allen Wittenauer added a comment - Ha. I beat it.
        Hide
        aw Allen Wittenauer added a comment -

        Thanks!

        Committed to trunk. I'm not going to bother waiting for Jenkins given that it probably won't even hit this code path.

        Show
        aw Allen Wittenauer added a comment - Thanks! Committed to trunk. I'm not going to bother waiting for Jenkins given that it probably won't even hit this code path.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 0s Docker mode activated.
        -1 patch 0m 5s HADOOP-13964 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.



        Subsystem Report/Notes
        JIRA Issue HADOOP-13964
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12846429/HADOOP-13964.00.patch
        Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/11461/console
        Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 0s Docker mode activated. -1 patch 0m 5s HADOOP-13964 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help. Subsystem Report/Notes JIRA Issue HADOOP-13964 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12846429/HADOOP-13964.00.patch Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/11461/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        andrew.wang Andrew Wang added a comment -

        +1 AFAIK we don't use these in CDH either

        Show
        andrew.wang Andrew Wang added a comment - +1 AFAIK we don't use these in CDH either
        Hide
        aw Allen Wittenauer added a comment -

        -00:

        • initial patch
        Show
        aw Allen Wittenauer added a comment - -00: initial patch

          People

          • Assignee:
            aw Allen Wittenauer
            Reporter:
            aw Allen Wittenauer
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development