Rename event batch timeout to event batch dropped.

remove unused function
gojek · Apr 22, 2024 · b696976 · b696976
1 parent 6452563
commit b696976
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 19 deletions.
diff --git a/Sources/Clickstream/NetworkManager/Core/RetryMechanism.swift b/Sources/Clickstream/NetworkManager/Core/RetryMechanism.swift
@@ -383,7 +383,7 @@ extension DefaultRetryMechanism {
                 persistence.deleteOne(eventRequest.guid)
                 #if TRACKER_ENABLED
                 if Tracker.debugMode {
-                    let healthEvent = HealthAnalysisEvent(eventName: .ClickstreamEventBatchTimeout,
+                    let healthEvent = HealthAnalysisEvent(eventName: .ClickstreamEventBatchDropped,
                                                           eventBatchGUID: fetchedEventRequest.guid,
                                                           eventCount: eventRequest.eventCount)
                     Tracker.sharedInstance?.record(event: healthEvent)
@@ -460,22 +460,6 @@ extension DefaultRetryMechanism {
 }
 
 // MARK: - Track Clickstream health.
-extension DefaultRetryMechanism {
-    func trackHealthEvents(eventRequest: EventRequest) {
-        #if TRACKER_ENABLED
-        if Tracker.debugMode {
-            guard eventRequest.eventType != Constants.EventType.instant else { return }
-
-            let healthEvent = HealthAnalysisEvent(eventName: .ClickstreamEventBatchSuccessAck,
-                                                  eventBatchGUID: eventRequest.guid,
-                                                  eventCount: eventRequest.eventCount)
-            Tracker.sharedInstance?.record(event: healthEvent)
-
-        }
-        #endif
-    }
-}
-
 extension DefaultRetryMechanism {
 
     func trackHealthAndPerformanceEvents(eventRequest: EventRequest, startTime: Date) {

diff --git a/Sources/Tracker/Utilities/TrackerConstants.swift b/Sources/Tracker/Utilities/TrackerConstants.swift
@@ -26,7 +26,7 @@ enum HealthEvents: String, Codable, CaseIterable {
     case ClickstreamEventBatchTriggerFailed = "Clickstream Event Batch Trigger Failed"
     case ClickstreamWriteToSocketFailed = "Clickstream Write to Socket Failed"
     case ClickstreamEventBatchErrorResponse = "Clickstream Event Batch Error response"
-    case ClickstreamEventBatchTimeout = "Clickstream Event Batch Timeout"
+    case ClickstreamEventBatchDropped = "Clickstream Event Batch Dropped"
 
     case ClickstreamConnectionSuccess = "Clickstream Connection Success"
     case ClickstreamConnectionFailure = "Clickstream Connection Failure"
@@ -80,5 +80,5 @@ public struct TrackerConstant {
         case aggregate = "aggregate"
     }
 
-    static let InstantEvents: [HealthEvents] = [.ClickstreamEventBatchTimeout, .ClickstreamConnectionSuccess, .ClickstreamConnectionFailure, .ClickstreamEventBatchErrorResponse, .ClickstreamWriteToSocketFailed, .ClickstreamEventBatchTriggerFailed]
+    static let InstantEvents: [HealthEvents] = [.ClickstreamEventBatchDropped, .ClickstreamConnectionSuccess, .ClickstreamConnectionFailure, .ClickstreamEventBatchErrorResponse, .ClickstreamWriteToSocketFailed, .ClickstreamEventBatchTriggerFailed]
 }