----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3442/#review4294 -----------------------------------------------------------
This is very, very helpful. Thanks. I focused on the javadoc but threw in java api nits as well (from the perspective of a mesos library maintainer). The javadoc should really follow http://www.oracle.com/technetwork/java/javase/documentation/index-137868.html, but the most glaring omissions are @parameter,@return docs. src/java/src/org/apache/mesos/Scheduler.java <https://reviews.apache.org/r/3442/#comment9675> The driver parameter in all methods other than register deserves explanation - must I use the it in the scope of the callback or is it always the same as the regsitered or ...? src/java/src/org/apache/mesos/Scheduler.java <https://reviews.apache.org/r/3442/#comment9676> There appears to be no need fo indexed access - Collection if you want to provide size, but it seems Iterable is fine here. src/java/src/org/apache/mesos/SchedulerDriver.java <https://reviews.apache.org/r/3442/#comment9671> s/public // - these are redundant for interface members src/java/src/org/apache/mesos/SchedulerDriver.java <https://reviews.apache.org/r/3442/#comment9672> how does this stop relate to the overload? failover=true or false? src/java/src/org/apache/mesos/SchedulerDriver.java <https://reviews.apache.org/r/3442/#comment9673> Is Iterable enough for the impl? - if so prefer it. - John On 2012-01-10 06:48:03, Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3442/ > ----------------------------------------------------------- > > (Updated 2012-01-10 06:48:03) > > > Review request for mesos, Andy Konwinski, Charles Reiss, Matei Zaharia, John > Sirois, and Vinod Kone. > > > Summary > ------- > > Provides initial documentation to the public interfaces (thanks to Vinod Kone > for pairing with me on this). > > > This addresses bug MESOS-45. > https://issues.apache.org/jira/browse/MESOS-45 > > > Diffs > ----- > > include/mesos/executor.hpp d883f1a > include/mesos/mesos.proto 77a9067 > include/mesos/scheduler.hpp 93dda98 > src/java/src/org/apache/mesos/Executor.java 5ad8cee > src/java/src/org/apache/mesos/ExecutorDriver.java be01d21 > src/java/src/org/apache/mesos/MesosExecutorDriver.java 399c8b8 > src/java/src/org/apache/mesos/MesosSchedulerDriver.java 04809a6 > src/java/src/org/apache/mesos/Scheduler.java 63a06fe > src/java/src/org/apache/mesos/SchedulerDriver.java 23a246c > src/python/src/mesos.py 5fc60e2 > > Diff: https://reviews.apache.org/r/3442/diff > > > Testing > ------- > > > Thanks, > > Benjamin > >
