Integrated in Hadoop-Common-trunk #693 (See https://builds.apache.org/hudson/job/Hadoop-Common-trunk/693/)
HADOOP-7137. Remove hod contrib docs. Contributed by Nigel Daley

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

• /hadoop/common/trunk/src/docs/src/documentation/content/xdocs/site.xml
• /hadoop/common/trunk/src/docs/src/documentation/content/xdocs/hod_scheduler.xml

Integrated in Hadoop-Common-22-branch #52 (See https://builds.apache.org/hudson/job/Hadoop-Common-22-branch/52/)
HADOOP-7137. svn merge -c 1124456 from trunk

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

• /hadoop/common/branches/branch-0.22/src/docs
• /hadoop/common/branches/branch-0.22/src/docs/src/documentation/content/xdocs/hod_scheduler.xml
• /hadoop/common/branches/branch-0.22
• /hadoop/common/branches/branch-0.22/src/test/core
• /hadoop/common/branches/branch-0.22/src/test/core/org/apache/hadoop/io/TestSequenceFile.java
• /hadoop/common/branches/branch-0.22/CHANGES.txt
• /hadoop/common/branches/branch-0.22/src/java
• /hadoop/common/branches/branch-0.22/src/docs/src/documentation/content/xdocs/site.xml

Integrated in Hadoop-Common-trunk-Commit #609 (See https://builds.apache.org/hudson/job/Hadoop-Common-trunk-Commit/609/)
HADOOP-7137. Remove hod contrib docs. Contributed by Nigel Daley

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

• /hadoop/common/trunk/src/docs/src/documentation/content/xdocs/site.xml
• /hadoop/common/trunk/src/docs/src/documentation/content/xdocs/hod_scheduler.xml

I've committed this to branch 22 and trunk. Thanks Nigel!

+1

I built and verified the generated docs.

A patch to also remove HOD english docs.

Oops, I mean I closed as won't fix all the hod Jira's. I didn't actually removed the Jira component.

Just removed contrib/hod from Jira and add to the Attic wiki: http://wiki.apache.org/hadoop/Attic

Integrated in Hadoop-Common-22-branch #49 (See https://builds.apache.org/hudson/job/Hadoop-Common-22-branch/49/)

Integrated in Hadoop-Common-trunk-Commit #601 (See https://builds.apache.org/hudson/job/Hadoop-Common-trunk-Commit/601/)

Integrated in Hadoop-Common-trunk #688 (See https://builds.apache.org/hudson/job/Hadoop-Common-trunk/688/)
HADOOP-7137. Remove hod contrib. Contributed by Nigel Daley

I should have. I didn't because most of the change is done via svn remove instead of applying the patch, so I ran the full tarball build, but waiting for Hudson would have caught HADOOP-7291 (the required change to test-patch.sh).

Why not wait for Hudson/Jenkins?

This commit removed test-contrib which is run by Hudson in PreCommit-HADOOP-Build:

[exec] ======================================================================
[exec] ======================================================================
[exec] Running contrib tests.
[exec] ======================================================================
[exec] ======================================================================
[exec]
[exec]
[exec] /bin/kill -9 12446
[exec] kill: No such process
[exec] /homes/hudson/tools/ant/latest/bin/ant -Dversion=1102861_HADOOP-7290_PATCH-12479160 -Declipse.home=/homes/hudson/tools/eclipse/latest -Dpython.home=/homes/hudson/tools/python/latest -DHadoopPatchProcess= -Dtest.junit.output.format=xml -Dtest.output=no test-contrib
[exec] Buildfile: build.xml
[exec]
[exec] BUILD FAILED
[exec] Target "test-contrib" does not exist in the project "Hadoop-Common".
[exec]
[exec] Total time: 0 seconds

I've committed this to trunk and branch 22. Thanks Nigel!

Tested via ant tar and ant package instead of Hudson since most of the change was done via svn remove (the patch only applies to build.xml).

+1

lgtm

-1 overall. Here are the results of testing the latest attachment
http://issues.apache.org/jira/secure/attachment/12477191/HADOOP-7137.patch
against trunk revision 1095958.

+1 @author. The patch does not contain any @author tags.

+1 tests included. The patch appears to include 80 new or modified tests.

-1 patch. The patch command could not apply the patch.

Console output: https://builds.apache.org/hudson/job/PreCommit-HADOOP-Build/373//console

This message is automatically generated.

This incorporates HADOOP-7138 and removes contrib from build and test targets.

Updated patch, ready for review.

Allen removed his -1. Reopening this.

We couldn't reach consensus on this so we won't be removing HOD for now.

sad but +1 on removing it from builds.

For now, we could remove it from builds etc.

Depends whether this falls under "Code Change" or "Adoption of New Codebase". The vote was clearly a question for the PMC. I'm fine either way.

This can't be committed until consensus is reached and Allen has -1'ed it.