----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53741/#review155931 -----------------------------------------------------------
src/webui/master/static/js/app.js (lines 7 - 8) <https://reviews.apache.org/r/53741/#comment226017> Let's move it after `.when('/'` src/webui/master/static/js/app.js (line 291) <https://reviews.apache.org/r/53741/#comment226019> Could not get the point why need this, I think `if (scope.columnKey === key) {` have cover this? src/webui/master/static/js/controllers.js (lines 458 - 459) <https://reviews.apache.org/r/53741/#comment226022> I think we could remove `'$routeParams', '$http', '$q', '$timeout', 'top'` here? src/webui/master/static/js/controllers.js (line 461) <https://reviews.apache.org/r/53741/#comment226027> We could send request to the leading master directly like what we do in `pollMetrics`. However, this require we public the `leadingMasterURL` method. May you mind add a ``` TODO(haosdent): Send requests to the leading master directly once `leadingMasterURL` is public. ``` here for me? src/webui/master/static/js/controllers.js (line 470) <https://reviews.apache.org/r/53741/#comment226020> Add a blank line below. - haosdent huang On Nov. 14, 2016, 9:27 p.m., Tomasz Janiszewski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53741/ > ----------------------------------------------------------- > > (Updated Nov. 14, 2016, 9:27 p.m.) > > > Review request for mesos, haosdent huang and Joseph Wu. > > > Bugs: MESOS-6443 > https://issues.apache.org/jira/browse/MESOS-6443 > > > Repository: mesos > > > Description > ------- > > Create new page with Maintenance schedule. Schedule is downloaded on > page refresh. Schedule is not live like stats and tasks so there is > no need to poll it periodically. > Diable sorting when data-key is not defined in table header. > > > Diffs > ----- > > src/webui/master/static/index.html 6211892e7d689df9bf3b2a9071a76ad4c60d0485 > src/webui/master/static/js/app.js c32177aa23c962f2bdf03d98272fb6d21a565382 > src/webui/master/static/js/controllers.js > dd0cc810748a9bd378d9c6b15e9fe89b7c0f11d9 > src/webui/master/static/maintenance.html PRE-CREATION > > Diff: https://reviews.apache.org/r/53741/diff/ > > > Testing > ------- > > [Screenshot](https://issues.apache.org/jira/secure/attachment/12838845/mesos_webui_maintenance_schedule.png) > > Testing done maually on Ubuntu/Chrome. Perfomrance testing done with 500 > entires schedule generated with > [generate_schedule.py](https://gist.github.com/janisz/e4dcb001f19aa4b466f9112a6dd16853) > > > Thanks, > > Tomasz Janiszewski > >
