Skip to content

Commit 83a1cfc

Browse files
committed
[Draft] Change to swap instead of filter legacy session
1 parent cd880cb commit 83a1cfc

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

firebase-perf/src/main/java/com/google/firebase/perf/logging/FirebaseSessionsEnforcementCheck.kt

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ package com.google.firebase.perf.logging
1919
import com.google.firebase.perf.session.isLegacy
2020
import com.google.firebase.perf.v1.NetworkRequestMetric
2121
import com.google.firebase.perf.v1.TraceMetric
22+
import java.util.Collections
2223
import com.google.firebase.perf.v1.PerfSession as ProtoPerfSession
2324

2425
class FirebaseSessionsEnforcementCheck {
@@ -50,12 +51,12 @@ class FirebaseSessionsEnforcementCheck {
5051
}
5152

5253
private fun filterLegacySessions(sessions: List<ProtoPerfSession>): List<ProtoPerfSession> {
53-
val updatedSessions = sessions.filter { !it.sessionId.isLegacy() }
54-
return if(updatedSessions.isEmpty()) {
55-
assert(!enforcement) { "No session besides a legacy session present. "}
56-
sessions
57-
} else {
54+
return if (sessions.count() > 0 && sessions[0].sessionId.isLegacy()) {
55+
val updatedSessions = sessions.toMutableList<ProtoPerfSession>()
56+
Collections.swap(updatedSessions, 0, 1)
5857
updatedSessions
58+
} else {
59+
sessions
5960
}
6061
}
6162
}

0 commit comments

Comments
 (0)