Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2016-09-14 Thread Joris Van Remoortere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40545/#review148880 --- @Ian can you please re-open if this is still an issue? I could

Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2016-09-14 Thread Ian Babrou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40545/ --- (Updated Sept. 14, 2016, 9:16 a.m.) Review request for mesos. Bugs:

Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2015-11-24 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40545/#review107734 --- src/webui/master/static/js/services.js (line 130)

Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2015-11-24 Thread Ian Babrou
> On Nov. 24, 2015, 8:14 a.m., Klaus Ma wrote: > > src/webui/master/static/js/services.js, line 130 > > > > > > Do we know why timestamp is NaN? Because mesos slaves do not supply it in metrics. It's not NaN, it's

Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2015-11-24 Thread Klaus Ma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40545/#review107739 --- Ship it! Ship It! - Klaus Ma On Nov. 21, 2015, 1:01 a.m., Ian

Re: Review Request 40545: MESOS-3970: fix cpu usage in mesos ui

2015-11-24 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40545/#review107763 --- Bad review! Reviews applied: [] Error: No reviewers specified.