commit a822bc131e7bf9be5277cb4f8dcb4777a71c80b0
author Stanislav Zholnin <zholnin@google.com> Mon Oct 28 12:28:12 2019 +0000
committer Stanislav Zholnin <zholnin@google.com> Mon Oct 28 12:52:37 2019 +0000
tree d83daeb4d344e0f38cb8aeaefec0cc1d4a916baa
parent a24da2986b73aeba81f06ea276102e0667700e67

Change AppOps collection to include historical appops since beginning of time.
Running totals are expected to work correctly with value metric.

Change-Id: If86ec8721a4abf5eed644b05d3fe8a419e050213
Test: make statsd_testdrive && ./out/host/linux-x86/bin/statsd_testdrive 10060

apex/statsd/service/java/com/android/server/stats/StatsCompanionService.java

diff --git a/apex/statsd/service/java/com/android/server/stats/StatsCompanionService.java b/apex/statsd/service/java/com/android/server/stats/StatsCompanionService.java
index 1a10753..5e5a04b 100644
--- a/apex/statsd/service/java/com/android/server/stats/StatsCompanionService.java
+++ b/apex/statsd/service/java/com/android/server/stats/StatsCompanionService.java
@@ -156,8 +156,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.PrintWriter;
-import java.time.Instant;
-import java.time.temporal.ChronoUnit;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.HashMap;
@@ -2071,9 +2069,7 @@
 
             CompletableFuture<HistoricalOps> ops = new CompletableFuture<>();
             HistoricalOpsRequest histOpsRequest =
-                    new HistoricalOpsRequest.Builder(
-                            Instant.now().minus(1, ChronoUnit.HOURS).toEpochMilli(),
-                            Long.MAX_VALUE).build();
+                    new HistoricalOpsRequest.Builder(0, Long.MAX_VALUE).build();
             appOps.getHistoricalOps(histOpsRequest, mContext.getMainExecutor(), ops::complete);
 
             HistoricalOps histOps = ops.get(EXTERNAL_STATS_SYNC_TIMEOUT_MILLIS,