Use performance-now lib instead of perf-hooks to calc perf (#26)

* Log Telemetry from the Email Report Task

* Fix telemetry logs

* Add perfhooks

* Use performance-now to calc performance numbers for telemetry

Author: saikrishnav

Tasks/emailReportTask/package.json

@@ -11,7 +11,8 @@
     "azure-pipelines-task-lib": "^2.8.0",
     "js2xmlparser": "^4.0.0",
     "nodemailer": "^6.3.0",
+    "performance-now": "^2.1.0",
     "reflect-metadata": "^0.1.13",
     "xslt-processor": "^0.11.5"
   }
-}
+}d

Tasks/emailReportTask/providers/restclients/RetryablePromise.ts

@@ -1,12 +1,12 @@
 import { TelemetryLogger } from "../../telemetry/TelemetryLogger";
-const { performance } = require('perf_hooks');
+const now = require('performance-now');
 
 export class RetryablePromise {
 
     private static defaultRetryCount: number = 3;
 
     public static async RetryAsync<T>(executor: () => Promise<T>, executorName: string, times: number = this.defaultRetryCount): Promise<T> {
-        const perfStart = performance.now();
+        const perfStart = now();
         let attemptNumber = 1;
         let lastError: Error;
         try {
@@ -31,7 +31,7 @@ export class RetryablePromise {
         finally {
             if (attemptNumber > 1) {
                 // Log time taken after all retries
-                TelemetryLogger.LogModulePerf(executorName, performance.now() - perfStart);
+                TelemetryLogger.LogModulePerf(executorName, now() - perfStart);
             }
         }
     }

Tasks/emailReportTask/telemetry/TelemetryLogger.ts

@@ -1,7 +1,7 @@
 import { ReportConfiguration } from "../config/ReportConfiguration";
 import { PipelineType } from "../config/pipeline/PipelineType";
 import { EnumUtils } from "../utils/EnumUtils";
-const { performance } = require('perf_hooks');
+const now = require("performance-now")
 
 export class TelemetryLogger {
   public static readonly TELEMETRY_LINE =
@@ -87,14 +87,14 @@ export class TelemetryLogger {
   }
 
   public static async InvokeWithPerfLogger<T>(executor: () => Promise<T>, executorName: string): Promise<T> {
-    const perfStart = performance.now();
+    const perfStart = now();
     let returnVal: T;
     try {
       returnVal = await executor();
     }
     finally {
       // Log time taken by the dataprovider
-      TelemetryLogger.LogModulePerf(executorName, performance.now() - perfStart);
+      TelemetryLogger.LogModulePerf(executorName, now() - perfStart);
     }
     return returnVal;
   }

package-lock.json

@@ -33,6 +33,7 @@
     "azure-pipelines-task-lib": "^2.8.0",
     "js2xmlparser": "^4.0.0",
     "nodemailer": "^6.3.0",
+    "performance-now": "^2.1.0",
     "reflect-metadata": "^0.1.13",
     "stats-lite": "^2.0.0",
     "ts-mockito": "^2.3.1",