Skip to content

Commit aeec4aa

Browse files
trasktrask
committed
Simplify Http TelemetryBuilder signatures
1 parent f5dbd14 commit aeec4aa

File tree

28 files changed

+74
-104
lines changed

28 files changed

+74
-104
lines changed

‎instrumentation-api-incubator/src/main/java/io/opentelemetry/instrumentation/api/incubator/builder/internal/DefaultHttpClientInstrumenterBuilder.java‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -50,17 +50,15 @@ public final class DefaultHttpClientInstrumenterBuilder<REQUEST, RESPONSE> {
5050

5151
private final List<AttributesExtractor<? super REQUEST, ? super RESPONSE>> additionalExtractors =
5252
new ArrayList<>();
53-
private Function<
54-
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
55-
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
53+
private Function<SpanStatusExtractor<REQUEST, RESPONSE>, SpanStatusExtractor<REQUEST, RESPONSE>>
5654
statusExtractorTransformer = Function.identity();
5755
private final HttpClientAttributesExtractorBuilder<REQUEST, RESPONSE>
5856
httpAttributesExtractorBuilder;
5957
private final HttpClientAttributesGetter<REQUEST, RESPONSE> attributesGetter;
6058
private final HttpSpanNameExtractorBuilder<REQUEST> httpSpanNameExtractorBuilder;
6159

6260
@Nullable private final TextMapSetter<REQUEST> headerSetter;
63-
private Function<SpanNameExtractor<? super REQUEST>, ? extends SpanNameExtractor<? super REQUEST>>
61+
private Function<SpanNameExtractor<REQUEST>, ? extends SpanNameExtractor<REQUEST>>
6462
spanNameExtractorTransformer = Function.identity();
6563
private boolean emitExperimentalHttpClientMetrics = false;
6664
private Consumer<InstrumenterBuilder<REQUEST, RESPONSE>> builderCustomizer = b -> {};
@@ -111,9 +109,7 @@ public DefaultHttpClientInstrumenterBuilder<REQUEST, RESPONSE> addAttributeExtra
111109

112110
@CanIgnoreReturnValue
113111
public DefaultHttpClientInstrumenterBuilder<REQUEST, RESPONSE> setStatusExtractor(
114-
Function<
115-
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
116-
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
112+
Function<SpanStatusExtractor<REQUEST, RESPONSE>, SpanStatusExtractor<REQUEST, RESPONSE>>
117113
statusExtractor) {
118114
this.statusExtractorTransformer = statusExtractor;
119115
return this;
@@ -180,7 +176,7 @@ public DefaultHttpClientInstrumenterBuilder<REQUEST, RESPONSE> setKnownMethods(
180176
/** Sets custom {@link SpanNameExtractor} via transform function. */
181177
@CanIgnoreReturnValue
182178
public DefaultHttpClientInstrumenterBuilder<REQUEST, RESPONSE> setSpanNameExtractor(
183-
Function<SpanNameExtractor<? super REQUEST>, ? extends SpanNameExtractor<? super REQUEST>>
179+
Function<SpanNameExtractor<REQUEST>, SpanNameExtractor<REQUEST>>
184180
spanNameExtractorTransformer) {
185181
this.spanNameExtractorTransformer = spanNameExtractorTransformer;
186182
return this;

‎instrumentation-api-incubator/src/main/java/io/opentelemetry/instrumentation/api/incubator/builder/internal/DefaultHttpServerInstrumenterBuilder.java‎

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -46,16 +46,14 @@ public final class DefaultHttpServerInstrumenterBuilder<REQUEST, RESPONSE> {
4646

4747
private final List<AttributesExtractor<? super REQUEST, ? super RESPONSE>> additionalExtractors =
4848
new ArrayList<>();
49-
private Function<
50-
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
51-
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
49+
private Function<SpanStatusExtractor<REQUEST, RESPONSE>, SpanStatusExtractor<REQUEST, RESPONSE>>
5250
statusExtractorTransformer = Function.identity();
5351
private final HttpServerAttributesExtractorBuilder<REQUEST, RESPONSE>
5452
httpAttributesExtractorBuilder;
5553
private final HttpSpanNameExtractorBuilder<REQUEST> httpSpanNameExtractorBuilder;
5654

5755
@Nullable private final TextMapGetter<REQUEST> headerGetter;
58-
private Function<SpanNameExtractor<? super REQUEST>, ? extends SpanNameExtractor<? super REQUEST>>
56+
private Function<SpanNameExtractor<REQUEST>, SpanNameExtractor<REQUEST>>
5957
spanNameExtractorTransformer = Function.identity();
6058
private final HttpServerRouteBuilder<REQUEST> httpServerRouteBuilder;
6159
private final HttpServerAttributesGetter<REQUEST, RESPONSE> attributesGetter;
@@ -109,9 +107,7 @@ public DefaultHttpServerInstrumenterBuilder<REQUEST, RESPONSE> addAttributesExtr
109107

110108
@CanIgnoreReturnValue
111109
public DefaultHttpServerInstrumenterBuilder<REQUEST, RESPONSE> setStatusExtractor(
112-
Function<
113-
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
114-
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
110+
Function<SpanStatusExtractor<REQUEST, RESPONSE>, SpanStatusExtractor<REQUEST, RESPONSE>>
115111
statusExtractor) {
116112
this.statusExtractorTransformer = statusExtractor;
117113
return this;
@@ -179,7 +175,7 @@ public DefaultHttpServerInstrumenterBuilder<REQUEST, RESPONSE> setKnownMethods(
179175
/** Sets custom {@link SpanNameExtractor} via transform function. */
180176
@CanIgnoreReturnValue
181177
public DefaultHttpServerInstrumenterBuilder<REQUEST, RESPONSE> setSpanNameExtractor(
182-
Function<SpanNameExtractor<? super REQUEST>, ? extends SpanNameExtractor<? super REQUEST>>
178+
Function<SpanNameExtractor<REQUEST>, SpanNameExtractor<REQUEST>>
183179
spanNameExtractorTransformer) {
184180
this.spanNameExtractorTransformer = spanNameExtractorTransformer;
185181
return this;

‎instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/semconv/http/HttpClientTelemetry.java‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1+
/*
2+
* Copyright The OpenTelemetry Authors
3+
* SPDX-License-Identifier: Apache-2.0
4+
*/
5+
16
package io.opentelemetry.instrumentation.api.semconv.http;
27

3-
public interface HttpClientTelemetry<REQUEST, RESPONSE> {
4-
}
8+
public interface HttpClientTelemetry<REQUEST, RESPONSE> {}

‎instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/semconv/http/HttpClientTelemetryBuilder.java‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
1-
package io.opentelemetry.instrumentation.api.semconv.http;
2-
3-
public interface HttpClientTelemetryBuilder<REQUEST, RESPONSE> {
1+
/*
2+
* Copyright The OpenTelemetry Authors
3+
* SPDX-License-Identifier: Apache-2.0
4+
*/
45

6+
package io.opentelemetry.instrumentation.api.semconv.http;
57

6-
}
8+
public interface HttpClientTelemetryBuilder<REQUEST, RESPONSE> {}

‎instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/semconv/http/HttpServerTelemetry.java‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,8 @@
1+
/*
2+
* Copyright The OpenTelemetry Authors
3+
* SPDX-License-Identifier: Apache-2.0
4+
*/
5+
16
package io.opentelemetry.instrumentation.api.semconv.http;
27

3-
public interface HttpServerTelemetry<REQUEST, RESPONSE> {
4-
}
8+
public interface HttpServerTelemetry<REQUEST, RESPONSE> {}

‎instrumentation-api/src/main/java/io/opentelemetry/instrumentation/api/semconv/http/HttpServerTelemetryBuilder.java‎

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
/*
2+
* Copyright The OpenTelemetry Authors
3+
* SPDX-License-Identifier: Apache-2.0
4+
*/
5+
16
package io.opentelemetry.instrumentation.api.semconv.http;
27

38
import io.opentelemetry.instrumentation.api.instrumenter.AttributesExtractor;
@@ -14,8 +19,8 @@ HttpServerTelemetryBuilder<REQUEST, RESPONSE> addAttributesExtractor(
1419

1520
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setStatusExtractor(
1621
Function<
17-
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
18-
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
22+
SpanStatusExtractor<? super REQUEST, ? super RESPONSE>,
23+
? extends SpanStatusExtractor<? super REQUEST, ? super RESPONSE>>
1924
statusExtractor);
2025

2126
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setCapturedRequestHeaders(
@@ -24,11 +29,10 @@ HttpServerTelemetryBuilder<REQUEST, RESPONSE> setCapturedRequestHeaders(
2429
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setCapturedResponseHeaders(
2530
List<String> responseHeaders);
2631

27-
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setKnownMethods(
28-
Set<String> knownMethods);
32+
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setKnownMethods(Set<String> knownMethods);
2933

30-
HttpServerTelemetryBuilder<REQUEST, RESPONSE>
31-
setEmitExperimentalHttpServerMetrics(boolean emitExperimentalHttpServerMetrics);
34+
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setEmitExperimentalHttpServerMetrics(
35+
boolean emitExperimentalHttpServerMetrics);
3236

3337
HttpServerTelemetryBuilder<REQUEST, RESPONSE> setSpanNameExtractor(
3438
Function<SpanNameExtractor<? super REQUEST>, ? extends SpanNameExtractor<? super REQUEST>>

‎instrumentation/apache-httpclient/apache-httpclient-4.3/library/src/main/java/io/opentelemetry/instrumentation/apachehttpclient/v4_3/ApacheHttpClientTelemetryBuilder.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ public ApacheHttpClientTelemetryBuilder setEmitExperimentalHttpClientMetrics(
105105
@CanIgnoreReturnValue
106106
public ApacheHttpClientTelemetryBuilder setSpanNameExtractor(
107107
Function<
108-
SpanNameExtractor<? super ApacheHttpClientRequest>,
109-
? extends SpanNameExtractor<? super ApacheHttpClientRequest>>
108+
SpanNameExtractor<ApacheHttpClientRequest>,
109+
SpanNameExtractor<ApacheHttpClientRequest>>
110110
spanNameExtractorTransformer) {
111111
builder.setSpanNameExtractor(spanNameExtractorTransformer);
112112
return this;

‎instrumentation/apache-httpclient/apache-httpclient-5.2/library/src/main/java/io/opentelemetry/instrumentation/apachehttpclient/v5_2/ApacheHttpClient5TelemetryBuilder.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -112,8 +112,8 @@ public ApacheHttpClient5TelemetryBuilder setEmitExperimentalHttpClientMetrics(
112112
@CanIgnoreReturnValue
113113
public ApacheHttpClient5TelemetryBuilder setSpanNameExtractor(
114114
Function<
115-
SpanNameExtractor<? super ApacheHttpClient5Request>,
116-
? extends SpanNameExtractor<? super ApacheHttpClient5Request>>
115+
SpanNameExtractor<ApacheHttpClient5Request>,
116+
SpanNameExtractor<ApacheHttpClient5Request>>
117117
spanNameExtractorTransformer) {
118118
builder.setSpanNameExtractor(spanNameExtractorTransformer);
119119
return this;

‎instrumentation/apache-httpclient/apache-httpclient-5.2/library/src/main/java/io/opentelemetry/instrumentation/apachehttpclient/v5_2/ApacheHttpClientTelemetryBuilder.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ public ApacheHttpClientTelemetryBuilder setEmitExperimentalHttpClientMetrics(
105105
@CanIgnoreReturnValue
106106
public ApacheHttpClientTelemetryBuilder setSpanNameExtractor(
107107
Function<
108-
SpanNameExtractor<? super ApacheHttpClientRequest>,
109-
? extends SpanNameExtractor<? super ApacheHttpClientRequest>>
108+
SpanNameExtractor<ApacheHttpClientRequest>,
109+
SpanNameExtractor<ApacheHttpClientRequest>>
110110
spanNameExtractorTransformer) {
111111
builder.setSpanNameExtractor(spanNameExtractorTransformer);
112112
return this;

‎instrumentation/armeria/armeria-1.3/library/src/main/java/io/opentelemetry/instrumentation/armeria/v1_3/ArmeriaClientTelemetryBuilder.java‎

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,8 @@ public final class ArmeriaClientTelemetryBuilder {
3838
@CanIgnoreReturnValue
3939
public ArmeriaClientTelemetryBuilder setStatusExtractor(
4040
Function<
41-
SpanStatusExtractor<? super ClientRequestContext, ? super RequestLog>,
42-
? extends SpanStatusExtractor<? super ClientRequestContext, ? super RequestLog>>
41+
SpanStatusExtractor<ClientRequestContext, RequestLog>,
42+
SpanStatusExtractor<ClientRequestContext, RequestLog>>
4343
statusExtractor) {
4444
builder.setStatusExtractor(statusExtractor);
4545
return this;
@@ -131,9 +131,7 @@ public ArmeriaClientTelemetryBuilder setEmitExperimentalHttpClientMetrics(
131131
/** Sets custom client {@link SpanNameExtractor} via transform function. */
132132
@CanIgnoreReturnValue
133133
public ArmeriaClientTelemetryBuilder setSpanNameExtractor(
134-
Function<
135-
SpanNameExtractor<? super ClientRequestContext>,
136-
? extends SpanNameExtractor<? super ClientRequestContext>>
134+
Function<SpanNameExtractor<ClientRequestContext>, SpanNameExtractor<ClientRequestContext>>
137135
clientSpanNameExtractor) {
138136
builder.setSpanNameExtractor(clientSpanNameExtractor);
139137
return this;

0 commit comments

Comments
 (0)