----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29539/#review66834 -----------------------------------------------------------
One high-level question below, or else LGTM src/master/contender.cpp <https://reviews.apache.org/r/29539/#comment110491> What guarantees that _url is a domain name? If that doesn't matter, then I don't understand the second argument to url(). src/master/detector.cpp <https://reviews.apache.org/r/29539/#comment110492> Same here. - Niklas Nielsen On Jan. 1, 2015, 8:46 p.m., Benjamin Hindman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29539/ > ----------------------------------------------------------- > > (Updated Jan. 1, 2015, 8:46 p.m.) > > > Review request for mesos and Niklas Nielsen. > > > Repository: mesos-git > > > Description > ------- > > See summary. > > > Diffs > ----- > > src/common/protobuf_utils.cpp 8ab5cdd80ecd499cb353141c1eaedaeb2a8b8b74 > src/master/contender.cpp c1bf82b621d6b46afe001acafe9ee53336726406 > src/master/detector.cpp 700eb9dde8e71648bacc00a82766634f77cf2d15 > src/master/master.cpp d6651e299ddb73bfdc1b126c474075db6cda8acd > src/sched/sched.cpp 4be08f12e126ac192a5247ec426a36610bb021d1 > src/scheduler/scheduler.cpp ff6ff115152c9278df6916fe9db432cbf127984e > src/slave/http.cpp d1cf8a68fab9a2df44f6c753683ad37fd4b1a1f9 > src/slave/slave.cpp 50b57819b55bdcdb9f49f20648199badc4d3f37b > src/tests/fetcher_tests.cpp 8c0b0757eb388f1684d8b94393983f1844a769a7 > src/tests/files_tests.cpp 9ad6db51873b96b3cd759523cef9748f6823fb7e > src/tests/gc_tests.cpp cb5dc5fa720bd2f344a9c7a3748dbe6f898705ca > src/tests/logging_tests.cpp 22a785e80304415ff2aacf76972fd58c74eb5c15 > src/tests/master_contender_detector_tests.cpp > d847a30d21b2a2980c6b7ceb62bbf61dc77487de > src/tests/master_tests.cpp 7f796be085bc0692f1412cec57468fe1ab0ee894 > src/tests/metrics_tests.cpp 7512b908ec1cbf80b74b768613a5f7925fa2249e > src/tests/monitor_tests.cpp 3b0261946405f9926742a680309880fac7480732 > > Diff: https://reviews.apache.org/r/29539/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Benjamin Hindman > >
