Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-24 Thread Jay Guo
> On March 22, 2016, 5:39 a.m., Jie Yu wrote: > > Instead of handling /containers endpoint in Slave, I would suggest we > > dispatch the request to ResourceMonitor. I will rename ResourceMonitor to > > ContainerMonitor, and we will be deprecating the /monitor/statistics > > endpoints in the fu

Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-23 Thread Jie Yu
> On March 22, 2016, 5:39 a.m., Jie Yu wrote: > > Instead of handling /containers endpoint in Slave, I would suggest we > > dispatch the request to ResourceMonitor. I will rename ResourceMonitor to > > ContainerMonitor, and we will be deprecating the /monitor/statistics > > endpoints in the fu

Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-22 Thread Jay Guo
> On March 22, 2016, 5:39 a.m., Jie Yu wrote: > > Instead of handling /containers endpoint in Slave, I would suggest we > > dispatch the request to ResourceMonitor. I will rename ResourceMonitor to > > ContainerMonitor, and we will be deprecating the /monitor/statistics > > endpoints in the fu

Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-22 Thread Jay Guo
> On March 22, 2016, 5:39 a.m., Jie Yu wrote: > > Instead of handling /containers endpoint in Slave, I would suggest we > > dispatch the request to ResourceMonitor. I will rename ResourceMonitor to > > ContainerMonitor, and we will be deprecating the /monitor/statistics > > endpoints in the fu

Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-22 Thread Mesos ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45014/#review124783 --- Patch looks great! Reviews applied: [45014] Passed command: expo

Re: Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-21 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45014/#review124735 --- Instead of handling /containers endpoint in Slave, I would suggest

Review Request 45014: Add /containers endpoint to return ResourceUsage.

2016-03-20 Thread Jay Guo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45014/ --- Review request for mesos and Jie Yu. Summary (updated) - Add /