diff options
author | Stefano Cianciulli <scianciulli@google.com> | 2024-05-10 10:49:46 +0000 |
---|---|---|
committer | Stefano Cianciulli <scianciulli@google.com> | 2024-05-10 10:49:46 +0000 |
commit | 2638c85b928af6659f6b7d3d0ef6fa735f791145 (patch) | |
tree | 6a3b0084393510e12cf8642e081a6f8d2834695e | |
parent | d76327758c7263601dc8177d0b3ab2afdd8985fa (diff) | |
download | proto_logging-2638c85b928af6659f6b7d3d0ef6fa735f791145.tar.gz |
Create new PreRebootDexoptJobEnded pushed atom in stats/atoms/art and reference it from atoms.proto
Bug: 336239721
Metrics Design Review: eldar/824633423
Test: presubmits
Merged-In: I3143615f0741c786354a1b1e6ba69ab576a05d7
Change-Id: I0e3eb50bca2f805f38e7b4655139219b1c770796
-rw-r--r-- | stats/atoms.proto | 1 | ||||
-rw-r--r-- | stats/atoms/art/background_extension_dexopt_atoms.proto | 46 |
2 files changed, 47 insertions, 0 deletions
diff --git a/stats/atoms.proto b/stats/atoms.proto index 170e6b87..944a46c9 100644 --- a/stats/atoms.proto +++ b/stats/atoms.proto @@ -1141,6 +1141,7 @@ message Atom { extensions 873; // LeScanAbused le_scan_abused extensions 874; // LeAdvStateChanged le_adv_state_changed extensions 875; // LeAdvErrorReported le_adv_error_reported + extensions 883; // PreRebootDexoptJobEnded pre_reboot_dexopt_job_ended extensions 9999; // Atom9999 atom_9999 // StatsdStats tracks platform atoms with ids up to 900. // Update StatsdStats::kMaxPushedAtomId when atom ids here approach that value. diff --git a/stats/atoms/art/background_extension_dexopt_atoms.proto b/stats/atoms/art/background_extension_dexopt_atoms.proto index 9a4e5030..383e2c1f 100644 --- a/stats/atoms/art/background_extension_dexopt_atoms.proto +++ b/stats/atoms/art/background_extension_dexopt_atoms.proto @@ -11,6 +11,7 @@ option java_multiple_files = true; extend Atom { optional BackgroundDexoptJobEnded background_dexopt_job_ended = 467 [(module) = "art"]; + optional PreRebootDexoptJobEnded prereboot_dexopt_job_ended = 883 [(module) = "art"]; } /** @@ -114,3 +115,48 @@ extend Atom { optional Pass pass = 8; } + +/** + * Logs when a pre-reboot dexopt job is ended. + * + * TODO(b/336239721): Mention the file containing the code logging this atom. + */ +message PreRebootDexoptJobEnded { + // Overall status of the job right before the reboot. + enum Status { + STATUS_UNKNOWN = 0; + STATUS_NOT_RUN = 1; + STATUS_FINISHED = 2; + STATUS_FAILED = 3; + STATUS_CANCELLED = 4; + } + optional Status status = 1; + + // Number of packages successfully optimized. + optional int32 optimized_package_count = 2; + // Number of packages failed to optimize. + optional int32 failed_package_count = 3; + // Number of packages skipped. + optional int32 skipped_package_count = 4; + // Total number of packages scanned. + optional int32 total_package_count = 5; + + // Duration of the job run (delta between when then job is run and when the + // job is finished or failed), in milliseconds. + optional int64 job_duration_millis = 6; + // Latency of the job run (delta between when the job is scheduled and when + // the job is finished or failed, including the time period between + // cancellations and reruns), in milliseconds. + optional int64 job_latency_millis = 7; + + // Number of packages that have Pre-reboot Dexopt artifacts after the + // reboot. Normally, this should be equal to `optimized_package_count`, but + // artifacts may be lost due to bugs. + optional int32 packages_with_artifacts_after_reboot_count = 8; + // Number of packages that have Pre-reboot Dexopt artifacts that are usable + // after the reboot. Normally, this should be equal to + // `packages_with_artifacts_after_reboot_count`, but artifacts may be + // invalid due to bugs. + optional int32 packages_with_artifacts_usable_after_reboot_count = 9; +} + |