----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6815/#review10864 -----------------------------------------------------------
Ship it! thanx for taking this on! src/flags/loader.hpp <https://reviews.apache.org/r/6815/#comment23400> new rule for spacing? src/java/jni/org_apache_mesos_Log.cpp <https://reviews.apache.org/r/6815/#comment23401> s/duration/time/ :))))) src/log/coordinator.cpp <https://reviews.apache.org/r/6815/#comment23402> s/0.0/0 src/log/coordinator.cpp <https://reviews.apache.org/r/6815/#comment23403> s/0.0/0 src/logging/logging.cpp <https://reviews.apache.org/r/6815/#comment23404> s/0.0/0 src/master/hierarchical_allocator_process.hpp <https://reviews.apache.org/r/6815/#comment23410> s/0.0/0 src/master/hierarchical_allocator_process.hpp <https://reviews.apache.org/r/6815/#comment23412> s/0.0/0 src/master/hierarchical_allocator_process.hpp <https://reviews.apache.org/r/6815/#comment23405> Filter should also just accept timeout third_party/libprocess/include/process/timer.hpp <https://reviews.apache.org/r/6815/#comment23406> s/because/as/ third_party/libprocess/src/process.cpp <https://reviews.apache.org/r/6815/#comment23407> s/30/30.0/ third_party/libprocess/src/process.cpp <https://reviews.apache.org/r/6815/#comment23409> s/30/30.0/ third_party/libprocess/src/tests.cpp <https://reviews.apache.org/r/6815/#comment23408> s/5/5.0/g - Vinod Kone On Aug. 29, 2012, 7:06 a.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6815/ > ----------------------------------------------------------- > > (Updated Aug. 29, 2012, 7:06 a.m.) > > > Review request for mesos and Vinod Kone. > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/exec/exec.cpp c9f87fa > src/flags/loader.hpp d7be7b9 > src/java/jni/org_apache_mesos_Log.cpp 3928b31 > src/java/jni/org_apache_mesos_state_ZooKeeperState.cpp 1f46e08 > src/linux/cgroups.hpp 5391c22 > src/linux/cgroups.cpp 9cedea5 > src/log/coordinator.cpp 838852f > src/log/log.hpp dcbf0b1 > src/log/replica.cpp e379d51 > src/logging/logging.cpp 6909b0b > src/master/constants.hpp 9214bab > src/master/frameworks_manager.cpp 6973389 > src/master/hierarchical_allocator_process.hpp 86c9b1e > src/master/master.cpp d3eab98 > src/mesos/main.cpp 747e98e > src/sched/sched.cpp a147e90 > src/slave/constants.hpp ab83972 > src/slave/flags.hpp 0c7917f > src/slave/gc.hpp 6704742 > src/slave/gc.cpp 9c01024 > src/slave/reaper.cpp 4acabad > src/slave/slave.cpp 4efd41e > src/tests/cgroups_tests.cpp 0c56481 > src/tests/fault_tolerance_tests.cpp 3cbab2b > src/tests/log_tests.cpp 85b2329 > src/tests/master_tests.cpp 36dd6a0 > src/zookeeper/group.cpp 8ae23f3 > src/zookeeper/zookeeper.cpp 621deaf > third_party/libprocess/include/process/delay.hpp f9f2c5e > third_party/libprocess/include/process/future.hpp cf0f522 > third_party/libprocess/include/process/latch.hpp f0afe78 > third_party/libprocess/include/process/process.hpp 9c90e76 > third_party/libprocess/include/process/timeout.hpp 4a1f113 > third_party/libprocess/include/process/timer.hpp 72f7711 > third_party/libprocess/include/stout/duration.hpp PRE-CREATION > third_party/libprocess/src/latch.cpp 2f9135d > third_party/libprocess/src/process.cpp 2b2d521 > third_party/libprocess/src/tests.cpp 8f4b2ae > > Diff: https://reviews.apache.org/r/6815/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
