----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18361/#review35306 -----------------------------------------------------------
Bad diff? I can't seem to view the "sched.cpp" diff. Error: The patch to 'src/sched/sched.cpp' didn't apply cleanly. The temporary files have been left in '/tmp/reviewboard.RGkfVR' for debugging purposes. `patch` returned: patching file /tmp/reviewboard.RGkfVR/tmpC62oNK Hunk #2 FAILED at 1070. 1 out of 4 hunks FAILED -- saving rejects to file /tmp/reviewboard.RGkfVR/tmpC62oNK-new.rej Also, I forgot to mention it in my last review, but can you deduce the default hostname based on the libprocess ip address instead of os::hostname()? That would be consistent with how we deduced the default hostname in master and slave. - Vinod Kone On Feb. 24, 2014, 5:24 p.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18361/ > ----------------------------------------------------------- > > (Updated Feb. 24, 2014, 5:24 p.m.) > > > Review request for mesos and Vinod Kone. > > > Bugs: MESOS-945 > https://issues.apache.org/jira/browse/MESOS-945 > > > Repository: mesos-git > > > Description > ------- > > - Added a field to FrameworkInfo in protobuf spec > - Added table fields in html of web ui that display framework's hostname for > running and terminated frameworks > - Added line to scheduler constructor that sets framework hostname. This is > just about the first centralized piece of code that every framework executes. > - Refactored scheduler constructor so that code duplication is avoided. > > > Diffs > ----- > > include/mesos/mesos.proto 69a4a60 > include/mesos/scheduler.hpp 2e4707e > src/master/http.cpp c941274 > src/sched/sched.cpp dcb3158 > src/slave/http.cpp 7c4cfba > src/webui/master/static/frameworks.html 8686b00 > > Diff: https://reviews.apache.org/r/18361/diff/ > > > Testing > ------- > > Built (with Clang) and ran local Mesos on Mac. Ran C++ and Java test > frameworks. Observed new hostname display in web UI. Built Mesos on Ubuntu > 13.10 with gcc 4.8.1. > > > Thanks, > > Bernd Mathiske > >
