> On Oct. 24, 2012, 11:23 p.m., Benjamin Hindman wrote: > > Just one high-level question: what happens when the webui GC's a directory. > > How does Files handle that, and what will you see in the webui? (Related, > > do we ever remove things from Files when the gc runs?)
No action is taken by Files. This means that the webui simply will show an error browsing this directory, since the directory doesn't exist. Removing things that get gc'ed is a little tricky, since we don't know the virtual name of the path at that point. Are you concerned about that? > On Oct. 24, 2012, 11:23 p.m., Benjamin Hindman wrote: > > src/webui/master/static/slave_executor.html, line 26 > > <https://reviews.apache.org/r/7642/diff/2/?file=178831#file178831line26> > > > > What happened here? Sorry about that, eclipse sometimes converts those unicode x's to that garbage. > On Oct. 24, 2012, 11:23 p.m., Benjamin Hindman wrote: > > src/webui/master/static/controllers.js, lines 542-544 > > <https://reviews.apache.org/r/7642/diff/2/?file=178826#file178826line542> > > > > I know you don't want me to say it, but the reuse of the SlaveCtrl does > > appear to be coming to a point where it's very very difficult to grep > > what's going on. Maybe not for this review, but it's probably time for a > > FrameworkCtrl and an ExecutorCtrl. Agreed, this got unwieldy, but since it's O(size(update() of HomeCtrl)) I'll punt with a TODO. - Ben ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7642/#review12733 ----------------------------------------------------------- On Oct. 25, 2012, 1:08 a.m., Ben Mahler wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7642/ > ----------------------------------------------------------- > > (Updated Oct. 25, 2012, 1:08 a.m.) > > > Review request for mesos, Benjamin Hindman and Vinod Kone. > > > Description > ------- > > -The slave now stores completed frameworks, executors, and tasks in memory. > -They are also exported via the state JSON endpoint. > -The webui has been updated accordingly. > > This is branched off of vinod's change here: > https://reviews.apache.org/r/7638/ > > Note that the JSON response sizes grow significantly due to this change, I'll > be following up with a change to pull in zlib and compress our HTTP Responses. > > > This addresses bug MESOS-263. > https://issues.apache.org/jira/browse/MESOS-263 > > > Diffs > ----- > > src/master/constants.hpp 4372f4d9d40b21701feae83ab56dc18afa735899 > src/master/http.cpp 9254859a0d4fddf775fd38d0482b4622dc767ab3 > src/master/master.hpp 146af017bbb6da9bd44acb53a4f1ee0ffbedd64f > src/master/master.cpp 82e4dc704e5c67ec178bd058934896328308d868 > src/messages/messages.proto 4e0538fe929f9091e5cdd4a1bb017d836df52a3e > src/slave/constants.hpp 4e28d30ebe3490f555a6dd43da9c6c1374ea4883 > src/slave/http.cpp d413c3c1bd9b6772d5ebf23fa8ae1bd5ae3870bb > src/slave/slave.hpp 343c353027dbd2a7c1be8cee99a1d59367169177 > src/slave/slave.cpp 5af7464aae17c00a0e707421982d7cb055aabc6c > src/webui/master/static/browse.html > e7e09a28ca9893ba049af100a257631d427ebfa1 > src/webui/master/static/controllers.js > 02a045d4a2ac01d468adc0885097f9191d7f1f08 > src/webui/master/static/framework.html > ecbf879b217ab51ea6f2531ebadd8dc09fd8ca54 > src/webui/master/static/frameworks.html > 4da01583eeb5bcd6ee5a36d4cbd58042bafcab15 > src/webui/master/static/home.html 84e9317b1b3bc857d6d6532adb25fd21a7691732 > src/webui/master/static/slave.html b96dfe95e08e79f683c622bf0bec96b9342c3be1 > src/webui/master/static/slave_executor.html > 33fa5f4ba3bacb5bb73849c1b1fa76a417351aa1 > src/webui/master/static/slave_framework.html > b43bbdcd79fab62d28e8d53737e3c4a083e0812d > > Diff: https://reviews.apache.org/r/7642/diff/ > > > Testing > ------- > > OSX + CentOS: make check > local run with long-lived-framework > > > Thanks, > > Ben Mahler > >
