feat: add isEnded on types.Span

open-telemetry · Feb 4, 2020 · b9a4cfd · b9a4cfd
1 parent 6d0bed3
commit b9a4cfd
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/packages/opentelemetry-api/src/trace/NoopSpan.ts b/packages/opentelemetry-api/src/trace/NoopSpan.ts
@@ -76,11 +76,19 @@ export class NoopSpan implements Span {
 
   // By default does nothing
   end(endTime?: TimeInput): void {}
+    this._ended = true;
+  }
 
   // isRecording always returns false for noopSpan.
   isRecording(): boolean {
     return false;
   }
+
+  isEnded(): boolean {
+    return this._ended;
+  }
+
+  private _ended: boolean = false;
 }
 
 export const NOOP_SPAN = new NoopSpan();
diff --git a/packages/opentelemetry-api/src/trace/span.ts b/packages/opentelemetry-api/src/trace/span.ts
@@ -100,4 +100,11 @@ export interface Span {
    * with the AddEvent operation and attributes using setAttributes.
    */
   isRecording(): boolean;
+
+  /**
+   * Returns if the span has been ended.
+   * 
+   * @returns true if the Span has been ended.
+   */
+  isEnded(): boolean;
 }
diff --git a/packages/opentelemetry-tracing/src/Span.ts b/packages/opentelemetry-tracing/src/Span.ts
@@ -172,6 +172,10 @@ export class Span implements types.Span, ReadableSpan {
     return true;
   }
 
+  isEnded(): boolean {
+    return this._ended;
+  }
+
   toReadableSpan(): ReadableSpan {
     return this;
   }