> On May 15, 2014, 10:52 p.m., Niklas Nielsen wrote: > > src/java/src/org/apache/mesos/Executor.java, lines 48-82 > > <https://reviews.apache.org/r/21080/diff/2/?file=582448#file582448line48> > > > > Thanks for expanding! > > However, I am not sure that it useful to list the protobuf field > > descriptions here. How about referring to the mesos.proto file? It is not > > all the descriptions that works out of context of the field being > > described. But maybe that's just me. > > > > If not referring to the .proto, can we maybe condense it a bit? > > > > And finally: > > s/ / /
Thanks Niklas. 1. Will wrap to 80, I was not sure about the wrapping for Java files thanks for pointing it out. (it is documented 70 for C++ files). 2. Right now we are not able to link to the protobuf file from the JDoc and I I think, for now, its good to provide this type of information (will condense it). 3. The unit of indentation should be 2 whitespaces then? - Bernardo ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21080/#review43169 ----------------------------------------------------------- On May 15, 2014, 8:56 p.m., Bernardo Gomez Palacio wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21080/ > ----------------------------------------------------------- > > (Updated May 15, 2014, 8:56 p.m.) > > > Review request for mesos and Niklas Nielsen. > > > Bugs: MESOS-1302 > https://issues.apache.org/jira/browse/MESOS-1302 > > > Repository: mesos-git > > > Description > ------- > > [MESOS-1259]:Enrich the Java Docs in the src/java files. -- > MesosSchedulerDriver.java > > > Diffs > ----- > > src/java/src/org/apache/mesos/Executor.java > cacb4e912cd44c5f6feb4a9920f4506e2399dcb7 > src/java/src/org/apache/mesos/ExecutorDriver.java > 56345dfb531492b32e032611e6fb8e68b234c5cc > src/java/src/org/apache/mesos/Log.java > d27e4f9e226f1aa5588cf75cb8c0a117dbf3015d > src/java/src/org/apache/mesos/MesosExecutorDriver.java > ce146cdb33b3c80ea43866d304b3f1ff1a3e7c00 > src/java/src/org/apache/mesos/MesosSchedulerDriver.java > 3fd1eb55468765cd9e9dceec53a653f6455000a3 > src/java/src/org/apache/mesos/Scheduler.java > 3043f60f408e11f5dae4d1c01b51840d4d289ba3 > src/java/src/org/apache/mesos/SchedulerDriver.java > ae2d915d9afb3d35d42a24b56115a386ea960154 > src/java/src/org/apache/mesos/state/State.java > d14d79b4850f677588c928803aba921030571fb8 > src/java/src/org/apache/mesos/state/Variable.java > 8e1ca69e673cedab8f05c53a376d6d421d09078b > > Diff: https://reviews.apache.org/r/21080/diff/ > > > Testing > ------- > > 1. ./bootstrap && mkdir build && cd build && ../configure && make > 2. cd src/java/target/apidocs > 3. Open index.html in your browser. i.e. open index.html > 4. Verify the Executor API. > > > Thanks, > > Bernardo Gomez Palacio > >