commit 31af6a9177129d0d1de920afa3acd8697cb80a35
author Tej Singh <singhtejinder@google.com> Thu May 21 17:12:41 2020 -0700
committer Tej Singh <singhtejinder@google.com> Thu May 21 17:12:41 2020 -0700
tree 89701df29c40cf089bcf44d956fcadbcea7ba835
parent a0ab5a1904dbfb75be7a90e151150aa772da5957

Update the default max pull delay to 30 seconds

Test: atest statsd_test
Bug: 154056912
Change-Id: If531db200a4d79eefaeefff60c86668f11d9439e

bin/src/guardrail/StatsdStats.h

diff --git a/bin/src/guardrail/StatsdStats.h b/bin/src/guardrail/StatsdStats.h
index 3d0eeb8..8587e14 100644
--- a/bin/src/guardrail/StatsdStats.h
+++ b/bin/src/guardrail/StatsdStats.h
@@ -160,7 +160,7 @@
     static const long kPullerCacheClearIntervalSec = 1;
 
     // Max time to do a pull.
-    static const int64_t kPullMaxDelayNs = 10 * NS_PER_SEC;
+    static const int64_t kPullMaxDelayNs = 30 * NS_PER_SEC;
 
     // Maximum number of pushed atoms statsd stats will track above kMaxPushedAtomId.
     static const int kMaxNonPlatformPushedAtoms = 100;

bin/src/statsd_config.proto

diff --git a/bin/src/statsd_config.proto b/bin/src/statsd_config.proto
index 2e6043d..72decf2 100644
--- a/bin/src/statsd_config.proto
+++ b/bin/src/statsd_config.proto
@@ -278,7 +278,7 @@
 
   optional int64 max_num_gauge_atoms_per_bucket = 11 [default = 10];
 
-  optional int32 max_pull_delay_sec = 13 [default = 10];
+  optional int32 max_pull_delay_sec = 13 [default = 30];
 
   optional bool split_bucket_for_app_upgrade = 14 [default = true];
 }
@@ -328,7 +328,7 @@
 
   optional bool skip_zero_diff_output = 14 [default = true];
 
-  optional int32 max_pull_delay_sec = 16 [default = 10];
+  optional int32 max_pull_delay_sec = 16 [default = 30];
 
   optional bool split_bucket_for_app_upgrade = 17 [default = true];