Improve logging and adda TODO

Author: tejasd

firebase-perf/src/main/java/com/google/firebase/perf/session/FirebasePerformanceSessionSubscriber.kt

@@ -16,8 +16,9 @@ class FirebasePerformanceSessionSubscriber(private val dataCollectionEnabled: Bo
     get() = SessionSubscriber.Name.PERFORMANCE
 
   override fun onSessionChanged(sessionDetails: SessionSubscriber.SessionDetails) {
-    AndroidLogger.getInstance().debug("AQS Session Changed: $sessionDetails")
     val perfSessionId = SessionManager.getInstance().perfSession().sessionId()
+    AndroidLogger.getInstance()
+      .debug("CFPRS AQS Session Changed: $sessionDetails, PerfSession: $perfSessionId")
 
     // There can be situations where a new [PerfSession] was created, but an AQS wasn't
     // available (during cold start).

firebase-perf/src/main/java/com/google/firebase/perf/session/PerfSession.java

@@ -20,6 +20,7 @@
 import androidx.annotation.Nullable;
 import androidx.annotation.VisibleForTesting;
 import com.google.firebase.perf.config.ConfigResolver;
+import com.google.firebase.perf.logging.AndroidLogger;
 import com.google.firebase.perf.util.Clock;
 import com.google.firebase.perf.util.Timer;
 import com.google.firebase.perf.v1.SessionVerbosity;
@@ -55,6 +56,7 @@ public PerfSession(String sessionId, Clock clock) {
     // SessionManagerKt verifies if this is an active session, and sets the AQS session ID.
     // The assumption is that new PerfSessions *should* be limited to either App Start, or through
     // AQS.
+    AndroidLogger.getInstance().debug("CFPRS PerfSession(): " + sessionId);
     FirebasePerformanceSessionSubscriber.Companion.getInstance().reportPerfSession(sessionId);
   }
 
@@ -176,9 +178,9 @@ public static com.google.firebase.perf.v1.PerfSession[] buildAndSort(
   /** If true, Session Gauge collection is enabled. */
   public static boolean shouldCollectGaugesAndEvents() {
     ConfigResolver configResolver = ConfigResolver.getInstance();
-
-    return configResolver.isPerformanceMonitoringEnabled()
-        && Math.random() < configResolver.getSessionsSamplingRate();
+    return true;
+    //    return configResolver.isPerformanceMonitoringEnabled()
+    //        && Math.random() < configResolver.getSessionsSamplingRate();
   }
 
   /**

firebase-perf/src/main/java/com/google/firebase/perf/session/SessionManager.java

@@ -68,6 +68,7 @@ public SessionManager(
     this.gaugeManager = gaugeManager;
     this.perfSession = perfSession;
     this.appStateMonitor = appStateMonitor;
+    AndroidLogger.getInstance().debug("CFPRS: SessionManager()");
   }
 
   /**
@@ -117,8 +118,6 @@ public void updatePerfSession(PerfSession perfSession) {
       return;
     }
 
-    AndroidLogger.getInstance().debug("Perf Session Changed: " + perfSession);
-
     this.perfSession = perfSession;
 
     synchronized (clients) {

firebase-perf/src/main/java/com/google/firebase/perf/session/gauges/GaugeManager.java

@@ -247,6 +247,7 @@ private void syncFlush(String sessionId, ApplicationProcessState appState) {
         FirebasePerformanceSessionSubscriber.Companion.getInstance()
             .getAqsMappedToPerfSession(sessionId);
     gaugeMetricBuilder.setSessionId(aqsSessionId);
+    AndroidLogger.getInstance().debug("CFPR syncFlush: " + sessionId + " AQS: " + aqsSessionId);
 
     transportManager.log(gaugeMetricBuilder.build(), appState);
   }
@@ -260,9 +261,13 @@ private void syncFlush(String sessionId, ApplicationProcessState appState) {
    * @return true if GaugeMetadata was logged, false otherwise.
    */
   public boolean logGaugeMetadata(String sessionId, ApplicationProcessState appState) {
+    // TODO(b/394127311): Based on logs, AQS session ID isn't available any time
+    // this is called. Adding a TODO to identify potential changes.
     String aqsSessionId =
         FirebasePerformanceSessionSubscriber.Companion.getInstance()
             .getAqsMappedToPerfSession(sessionId);
+    AndroidLogger.getInstance()
+        .debug("CFPR logGaugeMetadata: " + sessionId + " AQS: " + aqsSessionId);
     if (gaugeMetadataManager != null) {
       GaugeMetric gaugeMetric =
           GaugeMetric.newBuilder()