fix(sdk-logs): use Date instead of hrTime() for timestamps

open-telemetry · May 18, 2023 · 84125dc · 84125dc
1 parent 6d74a04
commit 84125dc
Showing 1 changed file with 3 additions and 11 deletions.
diff --git a/experimental/packages/sdk-logs/src/LogRecord.ts b/experimental/packages/sdk-logs/src/LogRecord.ts
@@ -21,7 +21,6 @@ import {
   timeInputToHrTime,
   isAttributeValue,
   InstrumentationScope,
-  hrTime,
 } from '@opentelemetry/core';
 import type { IResource } from '@opentelemetry/resources';
 
@@ -84,16 +83,9 @@ export class LogRecord implements ReadableLogRecord {
       context,
     } = logRecord;
 
-    if (timestamp && observedTimestamp) {
-      this.hrTime = timeInputToHrTime(timestamp);
-      this.hrTimeObserved = timeInputToHrTime(observedTimestamp);
-    } else {
-      const now = hrTime();
-      this.hrTime = timestamp ? timeInputToHrTime(timestamp) : now;
-      this.hrTimeObserved = observedTimestamp
-        ? timeInputToHrTime(observedTimestamp)
-        : now;
-    }
+    const now = Date.now();
+    this.hrTime = timeInputToHrTime(timestamp ?? now);
+    this.hrTimeObserved = timeInputToHrTime(observedTimestamp ?? now);
 
     if (context) {
       const spanContext = api.trace.getSpanContext(context);