Skip to content

Commit 92b85c1

Browse files
committed
Tighten TAsyncClient constructor matcher and drop unused transport binding
1 parent f4e6b67 commit 92b85c1

1 file changed

Lines changed: 1 addition & 4 deletions

File tree

instrumentation/thrift-0.13/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/thrift/v0_13/ThriftAsyncClientInstrumentation.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@
2929
import org.apache.thrift.async.AsyncMethodCallback;
3030
import org.apache.thrift.async.TAsyncClient;
3131
import org.apache.thrift.protocol.TProtocolFactory;
32-
import org.apache.thrift.transport.TNonblockingTransport;
3332

3433
class ThriftAsyncClientInstrumentation implements TypeInstrumentation {
3534

@@ -69,9 +68,7 @@ public void transform(TypeTransformer transformer) {
6968
public static class ConstructorAdvice {
7069
@Advice.AssignReturned.ToArguments(@ToArgument(0))
7170
@Advice.OnMethodEnter(suppress = Throwable.class, inline = false)
72-
public static TProtocolFactory onEnter(
73-
@Advice.Argument(0) TProtocolFactory protocolFactory,
74-
@Advice.Argument(2) TNonblockingTransport transport) {
71+
public static TProtocolFactory onEnter(@Advice.Argument(0) TProtocolFactory protocolFactory) {
7572
return new ClientProtocolDecorator.Factory(protocolFactory, getPropagators());
7673
}
7774
}

0 commit comments

Comments
 (0)