Skip to content

Commit

Permalink
Exclude jedis instrumentation from Indy-automigration (#9675)
Browse files Browse the repository at this point in the history
  • Loading branch information
JonasKunz authored Oct 16, 2023
1 parent 096d321 commit 6b0f479
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,14 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
public List<TypeInstrumentation> typeInstrumentations() {
return asList(new JedisConnectionInstrumentation(), new JedisInstrumentation());
}

@Override
public boolean isIndyModule() {
// java.lang.NoClassDefFoundError:
// io/opentelemetry/javaagent/instrumentation/jedis/JedisRequestContext
// at redis.clients.jedis.Jedis.flushAll(Jedis.java:367)
// at io.opentelemetry.javaagent.instrumentation.jedis.v1_4
// .JedisClientTest.setup(JedisClientTest.java:49)
return false;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -34,4 +34,14 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
public List<TypeInstrumentation> typeInstrumentations() {
return asList(new JedisConnectionInstrumentation(), new JedisInstrumentation());
}

@Override
public boolean isIndyModule() {
// java.lang.NoClassDefFoundError:
// io/opentelemetry/javaagent/instrumentation/jedis/JedisRequestContext
// at redis.clients.jedis.BinaryJedis.flushAll(BinaryJedis.java:595)
// at io.opentelemetry.javaagent.instrumentation.jedis.v3_0
// .Jedis30ClientTest.setup(Jedis30ClientTest.java:50)
return false;
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -30,4 +30,14 @@ public ElementMatcher.Junction<ClassLoader> classLoaderMatcher() {
public List<TypeInstrumentation> typeInstrumentations() {
return asList(new JedisConnectionInstrumentation(), new JedisInstrumentation());
}

@Override
public boolean isIndyModule() {
// java.lang.NoClassDefFoundError:
// io/opentelemetry/javaagent/instrumentation/jedis/JedisRequestContext
// at redis.clients.jedis.Jedis.set(Jedis.java:4613)
// at io.opentelemetry.javaagent.instrumentation.jedis
// .v4_0.Jedis40ClientTest.getCommand(Jedis40ClientTest.java:78)
return false;
}
}

0 comments on commit 6b0f479

Please sign in to comment.