Skip to content

Commit 27c9e8e

Browse files
otelbot[bot]laurit
andauthored
Review fixes for nats-2.17:library (#18220)
Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com> Co-authored-by: Lauri Tulmin <ltulmin@splunk.com>
1 parent 2ae0b57 commit 27c9e8e

3 files changed

Lines changed: 5 additions & 4 deletions

File tree

instrumentation/nats/nats-2.17/library/src/main/java/io/opentelemetry/instrumentation/nats/v2_17/OpenTelemetryConnection.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ final class OpenTelemetryConnection implements InvocationHandler {
2929
private final Instrumenter<NatsRequest, NatsRequest> producerInstrumenter;
3030
private final Instrumenter<NatsRequest, Void> consumerProcessInstrumenter;
3131

32-
public OpenTelemetryConnection(
32+
private OpenTelemetryConnection(
3333
Connection connection,
3434
Instrumenter<NatsRequest, NatsRequest> producerInstrumenter,
3535
Instrumenter<NatsRequest, Void> consumerProcessInstrumenter) {
@@ -38,7 +38,7 @@ public OpenTelemetryConnection(
3838
this.consumerProcessInstrumenter = consumerProcessInstrumenter;
3939
}
4040

41-
public static Connection wrap(
41+
static Connection wrap(
4242
Connection connection,
4343
Instrumenter<NatsRequest, NatsRequest> producerInstrumenter,
4444
Instrumenter<NatsRequest, Void> consumerProcessInstrumenter) {

instrumentation/nats/nats-2.17/library/src/main/java/io/opentelemetry/instrumentation/nats/v2_17/OpenTelemetryDispatcher.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,13 @@ final class OpenTelemetryDispatcher implements InvocationHandler {
2121
private final Dispatcher delegate;
2222
private final Instrumenter<NatsRequest, Void> consumerProcessInstrumenter;
2323

24-
public OpenTelemetryDispatcher(
24+
private OpenTelemetryDispatcher(
2525
Dispatcher delegate, Instrumenter<NatsRequest, Void> consumerProcessInstrumenter) {
2626
this.delegate = delegate;
2727
this.consumerProcessInstrumenter = consumerProcessInstrumenter;
2828
}
2929

30-
public static Dispatcher wrap(
30+
static Dispatcher wrap(
3131
Dispatcher delegate, Instrumenter<NatsRequest, Void> consumerProcessInstrumenter) {
3232
return (Dispatcher)
3333
Proxy.newProxyInstance(

instrumentation/nats/nats-2.17/metadata.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ semantic_conventions:
55
library_link: https://nats.io/
66
configurations:
77
- name: otel.instrumentation.messaging.experimental.capture-headers
8+
declarative_name: java.common.messaging.capture_headers/development
89
description: >
910
Enables capturing messaging headers as span attributes. Provide a comma-separated list of
1011
header names to capture.

0 commit comments

Comments
 (0)