Why is the docs target copying xml template files to build ? In case it is needed, shouldn't the target be overridden in capacity-scheduler's build file, rather than putting contrib specific operations in build.xml.
The link to conf/mapred-queues.xml.template in cluster_setup.xml seems wrong. It is pointing t mapred-queues.xml and not the template.
The docs target is to generate the default config files so that we can publish them to the site when 0.21 is release. See http://hadoop.apache.org/common/docs/current/core-default.html for an example which is referenced to from cluster_setup.html.
As suggested, I've moved the target to capacity-scheduler build file. Eventually, during release, we should commit the following:
- build/docs/mapred-queues.xml to "http://hadoop.apache.org/mapreduce/docs/current/mapred-queues.xml"
- build/docs/mapred-queues-capacity-scheduler.xml to "http://hadoop.apache.org/mapreduce/docs/current/mapred-queues-capacity-scheduler.xml"
- build/docs/capacity-scheduler-conf.html to "http://hadoop.apache.org/mapreduce/docs/current/capacity-scheduler-conf.html"
Queue refresh is also a scheduler specific supported feature. Should we call that out in the cluster-setup documentation ?
This was already done in the patch.
Included rest of the comments.