chore: allow parent span to be null

packages/opentelemetry-plugin-dns/src/dns.ts

@@ -116,7 +116,7 @@ export class DnsPlugin extends BasePlugin<Dns> {
       plugin._logger.debug('wrap lookup callback function and starts span');
       const name = utils.getOperationName('lookup');
       const span = plugin._startDnsSpan(name, {
-        parent: plugin._tracer.getCurrentSpan() || undefined,
+        parent: plugin._tracer.getCurrentSpan(),
         attributes: {
           [AttributeNames.PEER_HOSTNAME]: hostname,
         },

packages/opentelemetry-plugin-document-load/src/documentLoad.ts

@@ -121,16 +121,14 @@ export class DocumentLoad extends BasePlugin<unknown> {
     const metaElement = [...document.getElementsByTagName('meta')].find(
       e => e.getAttribute('name') === TRACE_PARENT_HEADER
     );
-    const serverContext =
-      parseTraceParent((metaElement && metaElement.content) || '') || undefined;
 
     const entries = this._getEntries();
 
     const rootSpan = this._startSpan(
       AttributeNames.DOCUMENT_LOAD,
       PTN.FETCH_START,
       entries,
-      { parent: serverContext }
+      { parent: parseTraceParent((metaElement && metaElement.content) || '') }
     );
     if (!rootSpan) {
       return;

packages/opentelemetry-plugin-grpc/src/grpc.ts

@@ -172,10 +172,9 @@ export class GrpcPlugin extends BasePlugin<grpc> {
               const self = this;
 
               const spanName = `grpc.${name.replace('/', '')}`;
-              const parentSpan = plugin._getSpanContext(call.metadata);
               const spanOptions: SpanOptions = {
                 kind: SpanKind.SERVER,
-                parent: parentSpan || undefined,
+                parent: plugin._getSpanContext(call.metadata),
               };
 
               plugin._logger.debug(
@@ -347,11 +346,10 @@ export class GrpcPlugin extends BasePlugin<grpc> {
       return function clientMethodTrace(this: grpcTypes.Client) {
         const name = `grpc.${original.path.replace('/', '')}`;
         const args = Array.prototype.slice.call(arguments);
-        const currentSpan = plugin._tracer.getCurrentSpan();
         const span = plugin._tracer
           .startSpan(name, {
             kind: SpanKind.CLIENT,
-            parent: currentSpan || undefined,
+            parent: plugin._tracer.getCurrentSpan(),
           })
           .setAttribute(AttributeNames.COMPONENT, GrpcPlugin.component);
         return plugin._makeGrpcClientRemoteCall(

packages/opentelemetry-plugin-postgres/opentelemetry-plugin-pg/src/utils.ts

@@ -52,7 +52,7 @@ function pgStartSpan(
   const jdbcString = getJDBCString(client.connectionParameters);
   return tracer.startSpan(name, {
     kind: SpanKind.CLIENT,
-    parent: tracer.getCurrentSpan() || undefined,
+    parent: tracer.getCurrentSpan(),
     attributes: {
       [AttributeNames.COMPONENT]: PostgresPlugin.COMPONENT, // required
       [AttributeNames.DB_INSTANCE]: client.connectionParameters.database, // required

packages/opentelemetry-plugin-redis/src/utils.ts

@@ -72,14 +72,12 @@ export const getTracedInternalSendCommand = (
     this: redisTypes.RedisClient & RedisPluginClientTypes,
     cmd?: RedisCommand
   ) {
-    const parentSpan = tracer.getCurrentSpan();
-
     // New versions of redis (2.4+) use a single options object
     // instead of named arguments
     if (arguments.length === 1 && typeof cmd === 'object') {
       const span = tracer.startSpan(`${RedisPlugin.COMPONENT}-${cmd.command}`, {
         kind: SpanKind.CLIENT,
-        parent: parentSpan || undefined,
+        parent: tracer.getCurrentSpan(),
         attributes: {
           [AttributeNames.COMPONENT]: RedisPlugin.COMPONENT,
           [AttributeNames.DB_STATEMENT]: cmd.command,

packages/opentelemetry-tracing/src/BasicTracer.ts

@@ -174,7 +174,7 @@ export class BasicTracer implements types.Tracer {
   }
 
   private _getParentSpanContext(
-    parent: types.Span | types.SpanContext | undefined
+    parent?: types.Span | types.SpanContext | null
   ): types.SpanContext | undefined {
     if (!parent) return undefined;
 

packages/opentelemetry-types/src/trace/SpanOptions.ts

@@ -46,7 +46,7 @@ export interface SpanOptions {
    * A parent `SpanContext` (or `Span`, for convenience) that the newly-started
    * span will be the child of.
    */
-  parent?: Span | SpanContext;
+  parent?: Span | SpanContext | null;
 
   /** A manually specified start time for the created `Span` object. */
   startTime?: number;