----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/18361/#review35164 -----------------------------------------------------------
Ship it! Ship It! include/mesos/mesos.proto <https://reviews.apache.org/r/18361/#comment65579> remove the trailing whitespace please - Dominic Hamon On Feb. 21, 2014, 9:49 a.m., Bernd Mathiske wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/18361/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2014, 9:49 a.m.) > > > Review request for mesos. > > > 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 > >
