----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13087/#review27716 -----------------------------------------------------------
partial review. src/cli/resolve.cpp <https://reviews.apache.org/r/13087/#comment53813> Is this true? src/local/local.cpp <https://reviews.apache.org/r/13087/#comment53815> Can we give them both the same name (Single or Standalone) for symmetry? src/master/contender.hpp <https://reviews.apache.org/r/13087/#comment53817> Kill this. src/master/contender.hpp <https://reviews.apache.org/r/13087/#comment53829> s/zk/master/ ? src/master/contender.hpp <https://reviews.apache.org/r/13087/#comment53818> Kill this. src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53819> reorder src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53821> virtual? src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53822> new line src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53823> LOG src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53824> new line src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53825> new line src/master/contender.cpp <https://reviews.apache.org/r/13087/#comment53826> LOG src/master/detector.hpp <https://reviews.apache.org/r/13087/#comment53827> When does the future fail/get discarded? src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53830> new line src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53828> new line. src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53831> s/Cannot/Failed to/ src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53832> s/INFO/WARNING/ ? src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53833> s/pid/expected/ ? src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53873> return in this case too? src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53835> s/previous/expected/ src/master/detector.cpp <https://reviews.apache.org/r/13087/#comment53872> pull this inside the if statement below? src/master/http.cpp <https://reviews.apache.org/r/13087/#comment53874> kill this now? src/master/master.hpp <https://reviews.apache.org/r/13087/#comment53875> s/lostCandidacy/lost/ ? src/master/master.cpp <https://reviews.apache.org/r/13087/#comment53877> why "somehow"? src/master/master.cpp <https://reviews.apache.org/r/13087/#comment53880> s/your/our/ s/. We/, we/ src/master/master.cpp <https://reviews.apache.org/r/13087/#comment53881> are you missing a return here? Also, is this expected? src/master/master.cpp <https://reviews.apache.org/r/13087/#comment53883> You should log who the leader is. src/sched/sched.cpp <https://reviews.apache.org/r/13087/#comment53884> s/Ignore/Ignoring/ src/sched/sched.cpp <https://reviews.apache.org/r/13087/#comment53886> why not CHECK_SOME here too? - Vinod Kone On Oct. 29, 2013, 5:02 p.m., Jiang Yan Xu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13087/ > ----------------------------------------------------------- > > (Updated Oct. 29, 2013, 5:02 p.m.) > > > Review request for mesos, Benjamin Hindman, Ben Mahler, and Vinod Kone. > > > Bugs: MESOS-496 > https://issues.apache.org/jira/browse/MESOS-496 > > > Repository: mesos-git > > > Description > ------- > > See summary. > > > Diffs > ----- > > include/mesos/scheduler.hpp fa1ffe8b24a054556593d31eeae3fd1ac1761fb5 > src/Makefile.am 0b32d74e43d21eb2f77b0abd8864ba5f8cf46214 > src/cli/resolve.cpp b05f5103ca898f74acb7dfe8f423ffc7e1872e09 > src/detector/detector.hpp 3aaebfe89f0b4bb5c64e7f4c8974d8b13bc6f6b1 > src/detector/detector.cpp 8d9f118757192ca38bcdb26663ed86effbcf3c9e > src/local/local.cpp e4b5ec5b3dfae6dd89746353d3caaa8d7e117840 > src/local/main.cpp 5995c538077dea301f231668b4b5f905e73e7b3b > src/master/contender.hpp PRE-CREATION > src/master/contender.cpp PRE-CREATION > src/master/detector.hpp PRE-CREATION > src/master/detector.cpp PRE-CREATION > src/master/http.cpp f2a535a002f07784db724f6fdedfb26b271d6327 > src/master/main.cpp 19fcb9f09d8fc03a0719aada7d216b575eb3069b > src/master/master.hpp 1eba03fc4fc2ae8b75603a241c431c7a4fe2249e > src/master/master.cpp 1147cc6ebd9a2d18c3b58fa103e6348f9d623438 > src/messages/messages.proto a5dded2c0f3f88a5f7d789ccde6e59f68880374b > src/sched/sched.cpp 3049096a7aeaea6c1063a1bbcef119517daa236c > src/slave/http.cpp 62fbb37a1924062543bf9db4229704bdef91601d > src/slave/main.cpp 750a12766bde64059bfd4635ea077cbd43cb4301 > src/slave/slave.hpp 68526f3fb01be6dc19d7e3c8e4ba8220b2c56ccb > src/slave/slave.cpp 06ef7860bf0d9531b50a504af0a46f6d2b900855 > src/tests/allocator_tests.cpp b0beb72273e9d44a407d0e83e6c18a541b212089 > src/tests/authentication_tests.cpp 48a9323d03416ad9ee25fc19838d89678ff613bc > src/tests/cluster.hpp 5621139df44e3e2ff330bdbe77fe3dda0165e143 > src/tests/fault_tolerance_tests.cpp > 3989e14eb926d7f9b5c7731aa0713a56fc3fb6e9 > src/tests/gc_tests.cpp 5459b78126d3607114ea171d3f76883f8355751c > src/tests/isolator_tests.cpp ab5b00aa7f12dbd299debc1a405ac50c59cec85c > src/tests/master_contender_detector_tests.cpp PRE-CREATION > src/tests/master_detector_tests.cpp > 06c586d29996cd599ae58dfd49cf1324aac0a6d6 > src/tests/master_tests.cpp bf790d24ff6b2fb199a04caebef523797b075e63 > src/tests/mesos.hpp ad39c095f9699fbe7958d52698fd2b0319e84eb9 > src/tests/mesos.cpp 10b3e5381af49cf07e0d0a6f457d646e6e6718ca > src/tests/slave_recovery_tests.cpp 2812f110925505258a291b72f0d6525ee56987c7 > > Diff: https://reviews.apache.org/r/13087/diff/ > > > Testing > ------- > > make check 100 times > > > Thanks, > > Jiang Yan Xu > >
