From a4354f989eaa433f1cab3f11f66609932a77fceb Mon Sep 17 00:00:00 2001 From: Alexander Dorokhine Date: Wed, 19 May 2021 14:00:19 -0700 Subject: [PATCH] Update statsd config to be mainline compatible. Instead of using the "framework" module, we use our own module for generating the atoms. Bug: 181787682 Test: Builds Change-Id: I626e2ddcad60d7a15c33b7a143b9e5d0ed0bb289 --- apex/appsearch/service/Android.bp | 23 ++++++++++++++----- .../appsearch/stats/PlatformLogger.java | 9 ++++---- 2 files changed, 21 insertions(+), 11 deletions(-) diff --git a/apex/appsearch/service/Android.bp b/apex/appsearch/service/Android.bp index 566a18fbfbc18..ad798c747d9cf 100644 --- a/apex/appsearch/service/Android.bp +++ b/apex/appsearch/service/Android.bp @@ -20,20 +20,31 @@ package { default_applicable_licenses: ["frameworks_base_license"], } +genrule { + name: "statslog-appsearch-java-gen", + tools: ["stats-log-api-gen"], + cmd: "$(location stats-log-api-gen) --java $(out) --module appsearch --javaPackage com.android.server.appsearch.stats --javaClass AppSearchStatsLog", + out: ["com/android/server/appsearch/stats/AppSearchStatsLog.java"], +} + java_library { name: "service-appsearch", installable: true, - srcs: ["java/**/*.java"], - libs: [ - "framework", - "framework-appsearch", - "services.core", - "services.usage", + srcs: [ + "java/**/*.java", + ":statslog-appsearch-java-gen", ], static_libs: [ "icing-java-proto-lite", "libicing-java", ], + libs: [ + "framework", + "framework-appsearch", + "framework-statsd.stubs.module_lib", + "services.core", + "services.usage", + ], required: [ "libicing", ], diff --git a/apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java b/apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java index 74aedbdf9056d..3027002f22115 100644 --- a/apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java +++ b/apex/appsearch/service/java/com/android/server/appsearch/stats/PlatformLogger.java @@ -30,7 +30,6 @@ import android.util.SparseIntArray; import com.android.internal.annotations.GuardedBy; import com.android.internal.annotations.VisibleForTesting; -import com.android.internal.util.FrameworkStatsLog; import com.android.server.appsearch.external.localstorage.AppSearchLogger; import com.android.server.appsearch.external.localstorage.stats.CallStats; import com.android.server.appsearch.external.localstorage.stats.InitializeStats; @@ -237,7 +236,7 @@ public final class PlatformLogger implements AppSearchLogger { String database = stats.getGeneralStats().getDatabase(); try { int hashCodeForDatabase = calculateHashCodeMd5(database); - FrameworkStatsLog.write(FrameworkStatsLog.APP_SEARCH_CALL_STATS_REPORTED, + AppSearchStatsLog.write(AppSearchStatsLog.APP_SEARCH_CALL_STATS_REPORTED, extraStats.mSamplingRatio, extraStats.mSkippedSampleCount, extraStats.mPackageUid, @@ -269,7 +268,7 @@ public final class PlatformLogger implements AppSearchLogger { String database = stats.getGeneralStats().getDatabase(); try { int hashCodeForDatabase = calculateHashCodeMd5(database); - FrameworkStatsLog.write(FrameworkStatsLog.APP_SEARCH_PUT_DOCUMENT_STATS_REPORTED, + AppSearchStatsLog.write(AppSearchStatsLog.APP_SEARCH_PUT_DOCUMENT_STATS_REPORTED, extraStats.mSamplingRatio, extraStats.mSkippedSampleCount, extraStats.mPackageUid, @@ -306,7 +305,7 @@ public final class PlatformLogger implements AppSearchLogger { String database = stats.getDatabase(); try { int hashCodeForDatabase = calculateHashCodeMd5(database); - FrameworkStatsLog.write(FrameworkStatsLog.APP_SEARCH_QUERY_STATS_REPORTED, + AppSearchStatsLog.write(AppSearchStatsLog.APP_SEARCH_QUERY_STATS_REPORTED, extraStats.mSamplingRatio, extraStats.mSkippedSampleCount, extraStats.mPackageUid, @@ -349,7 +348,7 @@ public final class PlatformLogger implements AppSearchLogger { mLastPushTimeMillisLocked = SystemClock.elapsedRealtime(); ExtraStats extraStats = createExtraStatsLocked(/*packageName=*/ null, CallStats.CALL_TYPE_INITIALIZE); - FrameworkStatsLog.write(FrameworkStatsLog.APP_SEARCH_INITIALIZE_STATS_REPORTED, + AppSearchStatsLog.write(AppSearchStatsLog.APP_SEARCH_INITIALIZE_STATS_REPORTED, extraStats.mSamplingRatio, extraStats.mSkippedSampleCount, extraStats.mPackageUid,