Modify telemetry to contain trigger time as property

Author: Kartik Raj

src/client/languageServer/watcher.ts

@@ -178,7 +178,9 @@ export class LanguageServerWatcher implements IExtensionActivationService, ILang
             // Start the language server.
             if (startupStopWatch) {
                 // It means that startup is triggering this code, track time it takes since startup to activate this code.
-                sendTelemetryEvent(EventName.LANGUAGE_SERVER_TRIGGER_DURATION, startupStopWatch.elapsedTime);
+                sendTelemetryEvent(EventName.LANGUAGE_SERVER_TRIGGER_DURATION, undefined, {
+                    triggerTime: startupStopWatch.elapsedTime,
+                });
             }
             await languageServerExtensionManager.startLanguageServer(lsResource, interpreter);
 

src/client/telemetry/index.ts

@@ -1330,10 +1330,15 @@ export interface IEventNamePropertyMapping {
      */
     /* __GDPR__
        "language_server_trigger_duration" : {
-          "duration" : { "classification": "SystemMetaData", "purpose": "FeatureInsight", "owner": "karrtikr", "isMeasurement": true }
+          "triggerTime" : { "classification": "SystemMetaData", "purpose": "FeatureInsight", "isMeasurement": true, "owner": "karrtikr" },
        }
      */
-    [EventName.LANGUAGE_SERVER_TRIGGER_DURATION]: unknown;
+    [EventName.LANGUAGE_SERVER_TRIGGER_DURATION]: {
+        /**
+         * Time it took to trigger language server startup.
+         */
+        triggerTime: number;
+    };
     /**
      * Telemetry event sent when starting Node.js server
      */