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

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.

For now, we could remove it from builds etc.

sad but +1 on removing it from builds.

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

Allen removed his -1. Reopening this.

Updated patch, ready for review.

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

-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.

+1

lgtm

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).

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

Why not wait for Hudson/Jenkins?

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).

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

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

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

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

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

A patch to also remove HOD english docs.

+1

I built and verified the generated docs.

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

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

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 #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