Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread weitao zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/ --- Review request for mesos. Repository: mesos Description --- mesos-ps is

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/#review85970 --- Patch looks great! Reviews applied: [34887] All tests passed. -

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/#review85979 --- Ship it! Ship It! - haosdent huang On June 1, 2015, 12:15 p.m.,

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread haosdent huang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/#review85978 --- src/cli/mesos-ps https://reviews.apache.org/r/34887/#comment137816

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34887/#review86037 --- Ship it! Ship It! - Ben Mahler On June 1, 2015, 12:15 p.m.,

Re: Review Request 34887: MESOS-2788: Fixed the statistics got by cmd *mesos ps* and webUI is mis-matching

2015-06-01 Thread haosdent huang
On June 1, 2015, 1:33 p.m., haosdent huang wrote: src/cli/mesos-ps, line 105 https://reviews.apache.org/r/34887/diff/1/?file=975642#file975642line105 How about add `bytes = float(bytes)` at the beginning. And don't need change `1024` to `1024.0` at below code. weitao zhou