> On Nov. 16, 2016, 8:18 a.m., haosdent huang wrote: > > src/webui/master/static/js/controllers.js, line 469 > > <https://reviews.apache.org/r/53741/diff/2/?file=1564539#file1564539line469> > > > > Any reason that remove `pollState();` here? > > Tomasz Janiszewski wrote: > This function was redundant since this page is not live updated.
Got it, thank you. :) - haosdent ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53741/#review156037 ----------------------------------------------------------- On Nov. 16, 2016, 7:54 p.m., Tomasz Janiszewski wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/53741/ > ----------------------------------------------------------- > > (Updated Nov. 16, 2016, 7:54 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 > >