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

Output javadoc inside the target directory

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0-alpha2
    • Fix Version/s: 3.0.0-alpha2
    • Component/s: documentation
    • Labels:
      None

      Description

      HADOOP-8500 cleaned up the javadoc build, but as a result we now have a few javadoc dirs being created outside target folders:

      	hadoop-common-project/hadoop-common/api/
      	hadoop-hdfs-project/hadoop-hdfs-client/api/
      	hadoop-hdfs-project/hadoop-hdfs/api/
      

      Thanks to Allen Wittenauer for finding this issue over on HADOOP-8500.

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10759 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10759/)
        HADOOP-13784. Output javadoc inside the target directory. (wang: rev dcc07ad8c8abf9feea569283b4a4a096d893168a)

        • (edit) hadoop-project-dist/pom.xml
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10759 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10759/ ) HADOOP-13784 . Output javadoc inside the target directory. (wang: rev dcc07ad8c8abf9feea569283b4a4a096d893168a) (edit) hadoop-project-dist/pom.xml
        Hide
        andrew.wang Andrew Wang added a comment -

        Thanks for reviewing Xiao, I've committed this to trunk.

        Show
        andrew.wang Andrew Wang added a comment - Thanks for reviewing Xiao, I've committed this to trunk.
        Hide
        xiaochen Xiao Chen added a comment -

        The patch makes sense to me. Just verified it locally the previously-lingering api dirs are gone. +1 let's get trunk fixed.

        Thanks for the quick fix Andrew, and sorry for the miss in HADOOP-8500!

        Show
        xiaochen Xiao Chen added a comment - The patch makes sense to me. Just verified it locally the previously-lingering api dirs are gone. +1 let's get trunk fixed. Thanks for the quick fix Andrew, and sorry for the miss in HADOOP-8500 !
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 23s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        -1 test4tests 0m 0s 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 mvninstall 7m 32s trunk passed
        +1 compile 0m 9s trunk passed
        +1 mvnsite 0m 10s trunk passed
        +1 mvneclipse 0m 10s trunk passed
        +1 javadoc 0m 9s trunk passed
        +1 mvninstall 0m 7s the patch passed
        +1 compile 0m 6s the patch passed
        +1 javac 0m 6s the patch passed
        +1 mvnsite 0m 7s the patch passed
        +1 mvneclipse 0m 7s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 xml 0m 1s The patch has no ill-formed XML file.
        +1 javadoc 0m 6s the patch passed
        +1 unit 0m 6s hadoop-project-dist in the patch passed.
        +1 asflicense 0m 14s The patch does not generate ASF License warnings.
        10m 4s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Issue HADOOP-13784
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12836483/HADOOP-13784.001.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit xml
        uname Linux 4a7601b2a538 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / bd7f591
        Default Java 1.8.0_101
        Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10957/testReport/
        modules C: hadoop-project-dist U: hadoop-project-dist
        Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10957/console
        Powered by Apache Yetus 0.4.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 23s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. -1 test4tests 0m 0s 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 mvninstall 7m 32s trunk passed +1 compile 0m 9s trunk passed +1 mvnsite 0m 10s trunk passed +1 mvneclipse 0m 10s trunk passed +1 javadoc 0m 9s trunk passed +1 mvninstall 0m 7s the patch passed +1 compile 0m 6s the patch passed +1 javac 0m 6s the patch passed +1 mvnsite 0m 7s the patch passed +1 mvneclipse 0m 7s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 xml 0m 1s The patch has no ill-formed XML file. +1 javadoc 0m 6s the patch passed +1 unit 0m 6s hadoop-project-dist in the patch passed. +1 asflicense 0m 14s The patch does not generate ASF License warnings. 10m 4s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Issue HADOOP-13784 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12836483/HADOOP-13784.001.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit xml uname Linux 4a7601b2a538 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / bd7f591 Default Java 1.8.0_101 Test Results https://builds.apache.org/job/PreCommit-HADOOP-Build/10957/testReport/ modules C: hadoop-project-dist U: hadoop-project-dist Console output https://builds.apache.org/job/PreCommit-HADOOP-Build/10957/console Powered by Apache Yetus 0.4.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        andrew.wang Andrew Wang added a comment -

        Patch attached. We now point the plugin so it outputs within the target directory.

        I ran the site build and browsed around, the Javadocs are still available at "/api".

        Show
        andrew.wang Andrew Wang added a comment - Patch attached. We now point the plugin so it outputs within the target directory. I ran the site build and browsed around, the Javadocs are still available at "/api".

          People

          • Assignee:
            andrew.wang Andrew Wang
            Reporter:
            andrew.wang Andrew Wang
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development