drop the TODO for review of db.namespace, this instr has no code owner :/

Author: trentm

packages/instrumentation-tedious/README.md

@@ -66,9 +66,6 @@ Attributes collected:
 | `net.peer.name`        | `server.address` | Remote hostname or similar.        |
 | `net.peer.port`        | `server.port`    | Remote port number.                |
 
-XXX Get review from owner on db.namespace and lengthy rec from https://opentelemetry.io/docs/specs/semconv/database/sql-server/ note `[1]`.
-
-
 ### Trace Context Propagation
 
 Database trace context propagation can be enabled by setting `enableTraceContextPropagation`to `true`.

packages/instrumentation-tedious/src/instrumentation.ts

@@ -247,6 +247,9 @@ export class TediousInstrumentation extends InstrumentationBase<TediousInstrumen
           attributes[ATTR_DB_SQL_TABLE] = request.table;
         }
         if (thisPlugin._dbSemconvStability & SemconvStability.STABLE) {
+          // The OTel spec for "db.namespace" discusses handling for connection
+          // to MSSQL "named instances". This isn't currently supported.
+          //    https://opentelemetry.io/docs/specs/semconv/database/sql-server/#:~:text=%5B1%5D%20db%2Enamespace
           attributes[ATTR_DB_NAMESPACE] = databaseName;
           attributes[ATTR_DB_SYSTEM_NAME] = DB_SYSTEM_NAME_VALUE_MICROSOFT_SQL_SERVER;
           attributes[ATTR_DB_QUERY_TEXT] = sql;