Skip to content

Commit 1d8ba85

Browse files
committed
Fix NullAway error in DbClientAttributesExtractor
Use getDbSystem() instead of getDbSystemName() since getDbSystem() is non-nullable and required by SemconvStability.stableDbSystemName().
1 parent 00ca4e2 commit 1d8ba85

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

instrumentation-api-incubator/src/main/java/io/opentelemetry/instrumentation/api/incubator/semconv/db/DbClientAttributesExtractor.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -77,14 +77,14 @@ static <REQUEST, RESPONSE> void onStartCommon(
7777
internalSet(
7878
attributes,
7979
DB_SYSTEM_NAME,
80-
SemconvStability.stableDbSystemName(getter.getDbSystemName(request)));
80+
SemconvStability.stableDbSystemName(getter.getDbSystem(request)));
8181
internalSet(attributes, DB_NAMESPACE, getter.getDbNamespace(request));
8282
internalSet(attributes, DB_QUERY_TEXT, getter.getDbQueryText(request));
8383
internalSet(attributes, DB_OPERATION_NAME, getter.getDbOperationName(request));
8484
internalSet(attributes, DB_QUERY_SUMMARY, getter.getDbQuerySummary(request));
8585
}
8686
if (SemconvStability.emitOldDatabaseSemconv()) {
87-
internalSet(attributes, DB_SYSTEM, getter.getDbSystemName(request));
87+
internalSet(attributes, DB_SYSTEM, getter.getDbSystem(request));
8888
internalSet(attributes, DB_USER, getter.getUser(request));
8989
internalSet(attributes, DB_NAME, getter.getDbNamespace(request));
9090
internalSet(attributes, DB_CONNECTION_STRING, getter.getConnectionString(request));

0 commit comments

Comments
 (0)