commit ec3ef1bc461a92153d597a8fbb3f683d908758e8
author Teferi Aleme <teferi@google.com> Wed May 24 22:27:00 2023 +0000
committer Teferi Aleme <teferi@google.com> Wed May 24 22:29:03 2023 +0000
tree 459587fab28ed598ce0d8fc059a13e1ec4ce7c89
parent fbdedb621e16c6925445d62e59661d931dd762d2

debug report: remove event report metrics logging

Bug: 284204782

Test: AdServicesServiceCoreUnitTests
Change-Id: I1d1b930d7b5e28c8b3c01937455235dcbcbe268c

adservices/service-core/java/com/android/adservices/service/measurement/reporting/DebugReportingJobHandler.java

diff --git a/adservices/service-core/java/com/android/adservices/service/measurement/reporting/DebugReportingJobHandler.java b/adservices/service-core/java/com/android/adservices/service/measurement/reporting/DebugReportingJobHandler.java
index 0765751..2cde7e0 100644
--- a/adservices/service-core/java/com/android/adservices/service/measurement/reporting/DebugReportingJobHandler.java
+++ b/adservices/service-core/java/com/android/adservices/service/measurement/reporting/DebugReportingJobHandler.java
@@ -16,8 +16,6 @@
 
 package com.android.adservices.service.measurement.reporting;
 
-import static com.android.adservices.service.stats.AdServicesStatsLog.AD_SERVICES_MEASUREMENT_REPORTS_UPLOADED__TYPE__EVENT;
-import static com.android.adservices.service.stats.AdServicesStatsLog.AD_SERVICES_MESUREMENT_REPORTS_UPLOADED;
 
 import android.adservices.common.AdServicesStatusUtils;
 import android.net.Uri;
@@ -26,8 +24,6 @@
 import com.android.adservices.data.enrollment.EnrollmentDao;
 import com.android.adservices.data.measurement.DatastoreManager;
 import com.android.adservices.data.measurement.IMeasurementDao;
-import com.android.adservices.service.stats.AdServicesLoggerImpl;
-import com.android.adservices.service.stats.MeasurementReportsStats;
 import com.android.internal.annotations.VisibleForTesting;
 
 import org.json.JSONArray;
@@ -60,8 +56,7 @@
 
         List<String> pendingDebugReportIdsInWindow = pendingDebugReports.get();
         for (String debugReportId : pendingDebugReportIdsInWindow) {
-            @AdServicesStatusUtils.StatusCode int result = performReport(debugReportId);
-            logReportingStats(result);
+            performReport(debugReportId);
         }
     }
 
@@ -124,14 +119,4 @@
         DebugReportSender debugReportSender = new DebugReportSender();
         return debugReportSender.sendReport(adTechDomain, debugReportPayload);
     }
-
-    private static void logReportingStats(int resultCode) {
-        AdServicesLoggerImpl.getInstance()
-                .logMeasurementReports(
-                        new MeasurementReportsStats.Builder()
-                                .setCode(AD_SERVICES_MESUREMENT_REPORTS_UPLOADED)
-                                .setType(AD_SERVICES_MEASUREMENT_REPORTS_UPLOADED__TYPE__EVENT)
-                                .setResultCode(resultCode)
-                                .build());
-    }
 }