Add option to defer ping lifetime metric persistence

glean-core/android/src/main/java/mozilla/telemetry/glean/Glean.kt

@@ -142,7 +142,8 @@ open class GleanInternalAPI internal constructor () {
             dataDir = this.gleanDataDir.path,
             packageName = applicationContext.packageName,
             uploadEnabled = uploadEnabled,
-            maxEvents = this.configuration.maxEvents
+            maxEvents = this.configuration.maxEvents,
+            delayPingLifetimeIO = false
         )
 
         // Start the migration from glean-ac, if needed.

glean-core/android/src/main/java/mozilla/telemetry/glean/config/Configuration.kt

@@ -17,12 +17,13 @@ import mozilla.telemetry.glean.net.PingUploader
  * **CAUTION**: This must match _exactly_ the definition on the Rust side.
  *  If this side is changed, the Rust side need to be changed, too.
  */
-@Structure.FieldOrder("dataDir", "packageName", "uploadEnabled", "maxEvents")
+@Structure.FieldOrder("dataDir", "packageName", "uploadEnabled", "maxEvents", "delayPingLifetimeIO")
 internal class FfiConfiguration(
     dataDir: String,
     packageName: String,
     uploadEnabled: Boolean,
-    maxEvents: Int? = null
+    maxEvents: Int? = null,
+    delayPingLifetimeIO: Boolean
 ) : Structure() {
     /**
      * Expose all structure fields as actual fields,
@@ -37,6 +38,8 @@ internal class FfiConfiguration(
     public var uploadEnabled: Byte = uploadEnabled.toByte()
     @JvmField
     public var maxEvents: IntByReference = if (maxEvents == null) IntByReference() else IntByReference(maxEvents)
+    @JvmField
+    public var delayPingLifetimeIO: Byte = delayPingLifetimeIO.toByte()
 
     init {
         // Force UTF-8 string encoding when passing strings over the FFI
@@ -64,6 +67,7 @@ data class Configuration internal constructor(
     val channel: String? = null,
     val userAgent: String = DEFAULT_USER_AGENT,
     val maxEvents: Int? = null,
+    val delayPingLifetimeIO: Boolean = false,
     val logPings: Boolean = DEFAULT_LOG_PINGS,
     // NOTE: since only simple object or strings can be made `const val`s, if the
     // default values for the lines below are ever changed, they are required
@@ -96,6 +100,7 @@ data class Configuration internal constructor(
         serverEndpoint = serverEndpoint,
         userAgent = DEFAULT_USER_AGENT,
         maxEvents = maxEvents,
+        delayPingLifetimeIO = false,
         logPings = DEFAULT_LOG_PINGS,
         httpClient = httpClient,
         pingTag = null,

glean-core/examples/sample.rs

@@ -23,6 +23,7 @@ fn main() {
         application_id: "org.mozilla.glean_core.example".into(),
         upload_enabled: true,
         max_events: None,
+        delay_ping_lifetime_io: false,
     };
     let mut glean = Glean::new(cfg).unwrap();
     glean.register_ping_type(&PingType::new("baseline", true, false));

glean-core/ffi/glean.h

@@ -80,6 +80,7 @@ typedef struct {
   FfiStr package_name;
   uint8_t upload_enabled;
   const int32_t *max_events;
+  uint8_t delay_ping_lifetime_io;
 } FfiConfiguration;
 
 /**

glean-core/ffi/src/lib.rs

@@ -101,6 +101,7 @@ pub struct FfiConfiguration<'a> {
     package_name: FfiStr<'a>,
     upload_enabled: u8,
     max_events: Option<&'a i32>,
+    delay_ping_lifetime_io: u8,
 }
 
 /// Convert the FFI-compatible configuration object into the proper Rust configuration object.
@@ -112,12 +113,14 @@ impl TryFrom<&FfiConfiguration<'_>> for glean_core::Configuration {
         let application_id = cfg.package_name.to_string_fallible()?;
         let upload_enabled = cfg.upload_enabled != 0;
         let max_events = cfg.max_events.filter(|&&i| i >= 0).map(|m| *m as usize);
+        let delay_ping_lifetime_io = cfg.delay_ping_lifetime_io != 0;
 
         Ok(Self {
             upload_enabled,
             data_path,
             application_id,
             max_events,
+            delay_ping_lifetime_io,
         })
     }
 }

glean-core/ios/Glean/GleanFfi.h

@@ -80,6 +80,7 @@ typedef struct {
   FfiStr package_name;
   uint8_t upload_enabled;
   const int32_t *max_events;
+  uint8_t delay_ping_lifetime_io;
 } FfiConfiguration;
 
 /**

glean-core/ios/Glean/Utils/Utils.swift

@@ -119,7 +119,8 @@ func withFfiConfiguration<R>(
         data_dir: dataDir,
         package_name: packageName,
         upload_enabled: uploadEnabled.toByte(),
-        max_events: maxEventsPtr
+        max_events: maxEventsPtr,
+        delay_ping_lifetime_io: false.toByte()
     )
     return body(cfg)
 }

glean-core/python/glean/_ffi.py

@@ -46,7 +46,11 @@ def _load_header(path: str) -> str:
 
 
 def make_config(
-    data_dir: Path, package_name: str, upload_enabled: bool, max_events: int
+    data_dir: Path,
+    package_name: str,
+    upload_enabled: bool,
+    max_events: int,
+    delay_ping_lifetime_io: bool = False,
 ) -> Any:
     """
     Make an `FfiConfiguration` object.
@@ -65,6 +69,7 @@ def make_config(
     cfg.package_name = package_name
     cfg.upload_enabled = upload_enabled
     cfg.max_events = max_events
+    cfg.delay_ping_lifetime_io = delay_ping_lifetime_io
 
     _global_weakkeydict[cfg] = (data_dir, package_name, max_events)