From 6a03680849cd1d920e3f260a149657edf2780ea0 Mon Sep 17 00:00:00 2001
From: Sergei Vorobyov <sergey.vorobyov@jetbrains.com>
Date: Fri, 2 Aug 2024 20:01:21 +0200
Subject: [PATCH] [Gradle|Sync|FUS] cleanup: use EventDataCollector for the
 complex Gradle sync fus event

GitOrigin-RevId: 623f8b3211cbb440eb28dbfb63cb8cf69035b0ff
---
 .../ExternalSystemSyncActionsCollector.kt     | 24 ++++++++++---------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/platform/external-system-impl/src/com/intellij/openapi/externalSystem/statistics/ExternalSystemSyncActionsCollector.kt b/platform/external-system-impl/src/com/intellij/openapi/externalSystem/statistics/ExternalSystemSyncActionsCollector.kt
index 6541969b32e5..a8764b5ae2c5 100644
--- a/platform/external-system-impl/src/com/intellij/openapi/externalSystem/statistics/ExternalSystemSyncActionsCollector.kt
+++ b/platform/external-system-impl/src/com/intellij/openapi/externalSystem/statistics/ExternalSystemSyncActionsCollector.kt
@@ -85,17 +85,19 @@ object ExternalSystemSyncActionsCollector : CounterUsagesCollector() {
   @JvmStatic
   fun logSyncFinished(project: Project?, activityId: Long, success: Boolean) {
     val nowTS = System.currentTimeMillis()
-
-    val (duration, parallelModelFetchEnabled) =
-      idToStartTS[activityId]
-        ?.let { Pair(nowTS - it.timestamp, it.parallelModelFetchEnabled) }
-      ?: Pair(-1L, false)
-
-    syncFinishedEvent.log(project,
-                          activityIdField.with(activityId),
-                          DurationMs.with(duration),
-                          isParallelModelFetch.with(parallelModelFetchEnabled),
-                          isSyncSuccessful.with(success))
+    syncFinishedEvent.log(project) {
+      add(activityIdField with activityId)
+      val syncDetails = idToStartTS[activityId]
+      if (syncDetails != null) {
+        add(DurationMs with nowTS - syncDetails.timestamp)
+        add(isParallelModelFetch with syncDetails.parallelModelFetchEnabled)
+      }
+      else {
+        add(DurationMs with -1)
+        add(isParallelModelFetch with false)
+      }
+      add(isSyncSuccessful with success)
+    }
   }
 
   @JvmStatic