> On 四月 11, 2016, 2:09 p.m., Klaus Ma wrote: > > Ship It! > > Klaus Ma wrote: > LGTM. Just wonder how to handle the input that did not follow our > expectation: > > // value,event,cgroup > // value,unit,event,cgroup > // value,unit,event,cgroup,running,ratio
Any format other than above is invalid :) - fan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44379/#review128117 ----------------------------------------------------------- On 四月 11, 2016, 7:01 a.m., fan du wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44379/ > ----------------------------------------------------------- > > (Updated 四月 11, 2016, 7:01 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-4705 > https://issues.apache.org/jira/browse/MESOS-4705 > > > Repository: mesos > > > Description > ------- > > Use tokens size to parse perf stat format. > > Co-authored with haosdent. > > > Diffs > ----- > > src/linux/perf.cpp 749e676aaf2ce639dd976f2b23e323300c6114c5 > > Diff: https://reviews.apache.org/r/44379/diff/ > > > Testing > ------- > > 1. {Found and Test} with Serenity, ema filter could get perf event statistics > correctly as expected. > 2. ./bin/mesos-tests.sh --gtest_filter=PerfEventIsolatorTest* > --log_dir=/tmp/mesos/ > > > Thanks, > > fan du > >
