diff options
| author | Olivier Gaillard <gaillard@google.com> | 2019-02-05 15:12:39 +0000 |
|---|---|---|
| committer | Olivier Gaillard <gaillard@google.com> | 2019-02-06 13:36:55 +0000 |
| commit | 9a5d359879b5e2f7e7c0730b04242d81dd10a5c8 (patch) | |
| tree | 0d12c13b5752f6ced7c1f26e930bc53a81efb2db /tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp | |
| parent | c5f11c40064e4ba3a3aa66d58e466ae54005c4ac (diff) | |
Adds the concept of invalid bucket.
Whenever a pull fails, we cannot compute a correct metric for this
bucket. Whether the failures happen at the beginning, in the middle or
at the end of the bucket, we cannot trust the data inside the bucket.
Let's say we have a metric with a screen on condition, a bucket size of
4 hours and we have the following events:
- h+0 bucket start -- screen is on -- data pull failed
- h+2 screen off -- pull succeed
- h+3 screen on -- pull succeed
- h+4 bucket end -- screen on -- pull succeed
The current logic will be wrong, it will ignore any data between h+0
and h+2. That timespan might be a huge contributor of the total bucket
value so this is wrong to keep any data from this bucket.
We also extend the concept of invalid buckets to other problems like a
pull being too long.
Bug: 123866830
Test: atest statsd_test
Change-Id: I300ab05cd7582cd2d7af9167de8d99b349071f0d
Diffstat (limited to 'tests/DynamicCodeLoggerIntegrationTests/src/cpp/test_executable.cpp')
0 files changed, 0 insertions, 0 deletions