----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12921/#review23902 -----------------------------------------------------------
hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java <https://reviews.apache.org/r/12921/#comment47727> How about: mapred.mesos.total.map.slots.minimum mapred.mesos.total.reduce.slots.minimum And for these added variables, can you add the default values in mapred-site.xml.patch and add a human friendly comment for others to know how to use the knob? hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java <https://reviews.apache.org/r/12921/#comment47728> It seems like the goal of the minimum here is to have a minimum amount of idle slots in the cluster? If so, we should likely update the configuration names to be: mapred.mesos.total.idle.map.slots.minimum mapred.mesos.total.idle.reduce.slots.minimum - Ben Mahler On July 25, 2013, 1:52 a.m., Brenden Matthews wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12921/ > ----------------------------------------------------------- > > (Updated July 25, 2013, 1:52 a.m.) > > > Review request for mesos. > > > Repository: mesos-git > > > Description > ------- > > Allowed specification of mininum slot count. > > To ensure Hadoop jobs begin promptly, we can specify a minimum number of > 'hot slots' to be available for use. This addresses the TaskTracker > spin up delay that exists with Hadoop on Mesos. This can be a nuisance > with lower latency applications, such as ad-hoc Hive queries. > > Review: https://reviews.apache.org/r/12921 > > > Diffs > ----- > > hadoop/mesos/src/java/org/apache/hadoop/mapred/MesosScheduler.java > d9a9dbb57b10a8450ef8d822a041fa8d716bae4c > > Diff: https://reviews.apache.org/r/12921/diff/ > > > Testing > ------- > > make check && cd hadoop && make hadoop-2.0.0-mr1-cdh4.2.1 > > > Thanks, > > Brenden Matthews > >
