> On 一月 14, 2016, 9:19 a.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [42288] > > > > Failed command: ./support/apply-review.sh -n -r 42288 > > > > Error: > > 2016-01-14 09:19:38 URL:https://reviews.apache.org/r/42288/diff/raw/ > > [612/612] -> "42288.patch" [1] > > Total errors found: 0 > > Checking 1 files > > Error: Commit message summary (the first line) must not exceed 72 > > characters.
Thanks?I have fix it, modify the summary as "Add timestamp to DockerContainerizer's ResourceStatistics" - Andy ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42288/#review114443 ----------------------------------------------------------- On 一月 21, 2016, 3:46 a.m., Andy Pang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42288/ > ----------------------------------------------------------- > > (Updated 一月 21, 2016, 3:46 a.m.) > > > Review request for mesos, Jie Yu and Timothy Chen. > > > Bugs: MESOS-4328 > https://issues.apache.org/jira/browse/MESOS-4328 > > > Repository: mesos > > > Description > ------- > > Docker container REST API /monitor/statistics.json output have no timestamp > field > > > Diffs > ----- > > src/slave/containerizer/docker.cpp da19975 > > Diff: https://reviews.apache.org/r/42288/diff/ > > > Testing > ------- > > make check > > > Thanks, > > Andy Pang > >
