Skip to content

Commit e21ded7

Browse files
committed
fix lints
Signed-off-by: Kavindu Dodanduwa <[email protected]>
1 parent a743319 commit e21ded7

File tree

2 files changed

+14
-17
lines changed

2 files changed

+14
-17
lines changed

Diff for: providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdGrpcInterceptor.java

+8-15
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,8 @@
1414
import javax.annotation.Nullable;
1515

1616
/**
17-
* FlagdGrpcInterceptor is an interceptor for grpc communication from java-sdk to flagd
18-
* <p>
19-
* <a href="https://github.com/open-telemetry/opentelemetry-java-docs">credits</a>
17+
* FlagdGrpcInterceptor is an interceptor for grpc communication from java-sdk to flagd.
18+
* <a href="https://github.com/open-telemetry/opentelemetry-java-docs">credits</a>
2019
*/
2120
final class FlagdGrpcInterceptor implements ClientInterceptor {
2221
private static final TextMapSetter<Metadata> SETTER = new Setter();
@@ -27,30 +26,24 @@ final class FlagdGrpcInterceptor implements ClientInterceptor {
2726
this.openTelemetry = openTelemetry;
2827
}
2928

30-
@Override
31-
public <ReqT, RespT> ClientCall<ReqT, RespT> interceptCall(
32-
MethodDescriptor<ReqT, RespT> methodDescriptor, CallOptions callOptions, Channel channel) {
29+
@Override public <ReqT, RespT> ClientCall<ReqT, RespT> interceptCall(MethodDescriptor<ReqT, RespT> methodDescriptor,
30+
CallOptions callOptions, Channel channel) {
3331

3432
final ClientCall<ReqT, RespT> call = channel.newCall(methodDescriptor, callOptions);
3533

3634
return new ForwardingClientCall.SimpleForwardingClientCall<ReqT, RespT>(call) {
37-
@Override
38-
public void start(Listener<RespT> responseListener, io.grpc.Metadata headers) {
39-
openTelemetry.getPropagators()
40-
.getTextMapPropagator()
41-
.inject(Context.current(), headers, SETTER);
42-
35+
@Override public void start(Listener<RespT> responseListener, io.grpc.Metadata headers) {
36+
openTelemetry.getPropagators().getTextMapPropagator().inject(Context.current(), headers, SETTER);
4337
super.start(responseListener, headers);
4438
}
4539
};
4640
}
4741

4842
/**
49-
* Setter implements TextMapSetter with carrier check
43+
* Setter implements TextMapSetter with carrier check.
5044
*/
5145
static class Setter implements TextMapSetter<Metadata> {
52-
@Override
53-
public void set(@Nullable Metadata carrier, String key, String value) {
46+
@Override public void set(@Nullable Metadata carrier, String key, String value) {
5447
if (carrier == null) {
5548
return;
5649
}

Diff for: providers/flagd/src/main/java/dev/openfeature/contrib/providers/flagd/FlagdProvider.java

+6-2
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,10 @@ public FlagdProvider() {
186186
fallBackToEnvOrDefault(MAX_EVENT_STREAM_RETRIES_ENV_VAR_NAME, DEFAULT_MAX_EVENT_STREAM_RETRIES));
187187
}
188188

189+
190+
/**
191+
* Create a new FlagdProvider instance with manual telemetry sdk.
192+
*/
189193
public FlagdProvider(OpenTelemetrySdk telemetrySdk) {
190194
this(
191195
buildServiceBlockingStub(telemetrySdk),
@@ -557,7 +561,7 @@ private <ValT extends Object, ReqT extends Message, ResT extends Message> Provid
557561
// run the referenced resolver method
558562
final ResT response;
559563

560-
if (tracer != null){
564+
if (tracer != null) {
561565
final Span span = tracer.spanBuilder("resolve")
562566
.setSpanKind(SpanKind.CLIENT)
563567
.startSpan();
@@ -567,7 +571,7 @@ private <ValT extends Object, ReqT extends Message, ResT extends Message> Provid
567571
} finally {
568572
span.end();
569573
}
570-
}else {
574+
} else {
571575
response = resolverRef.apply((ReqT) req);
572576
}
573577

0 commit comments

Comments
 (0)