----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23997/#review49033 -----------------------------------------------------------
src/master/contender.cpp <https://reviews.apache.org/r/23997/#comment85863> Master detector and contender already support --zk=file://, in the "else if (strings::startsWith(zk, "file://"))" branch below. Looks like the same logic exists in a couple of places it may be reasonable to pull it into "Try<URL> URL::parse(const std::string& url)" directly. src/master/detector.cpp <https://reviews.apache.org/r/23997/#comment85866> Ditto src/master/main.cpp <https://reviews.apache.org/r/23997/#comment85867> This is no longer a TODO as it's done now, right? - Jiang Yan Xu On July 29, 2014, 7:33 a.m., Ken Sipe wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/23997/ > ----------------------------------------------------------- > > (Updated July 29, 2014, 7:33 a.m.) > > > Review request for mesos, Adam B and Ben Mahler. > > > Bugs: MESOS-1635 > https://issues.apache.org/jira/browse/MESOS-1635 > > > Repository: mesos-git > > > Description > ------- > > Fixes MESOS-1635 "zk flag fails when specifying a file and the replicated > logs" > > replaced the current TODO with code :) The code is consistent with similar > types of checks elsewhere in code. > > > Diffs > ----- > > src/master/contender.cpp c1bf82b621d6b46afe001acafe9ee53336726406 > src/master/detector.cpp a34cc211941972e2d64802d5c7ed3f5026229744 > src/master/main.cpp 4d39c355eae948ede492d84de2e58a26756b781a > > Diff: https://reviews.apache.org/r/23997/diff/ > > > Testing > ------- > > The code change is simple and no unit test is provided. It compiles, passes > style check and is functional via manual check. > > > Thanks, > > Ken Sipe > >
