From 1e88e73668d699d657e24635b3d7b7c577e0d1ba Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 04:38:08 +0000 Subject: [PATCH 1/8] Review fixes for jdbc:testing Automated code review of instrumentation/jdbc/testing. --- .../AbstractJdbcInstrumentationTest.java | 33 +++++-------------- .../jdbc/testing/ProxyStatementFactory.java | 3 +- 2 files changed, 11 insertions(+), 25 deletions(-) diff --git a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java index bd8df4b2240e..08ec89d84e84 100644 --- a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java +++ b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java @@ -46,7 +46,6 @@ import io.opentelemetry.sdk.testing.assertj.TraceAssert; import java.beans.PropertyVetoException; import java.io.Closeable; -import java.io.IOException; import java.sql.CallableStatement; import java.sql.Connection; import java.sql.Driver; @@ -68,7 +67,6 @@ import org.assertj.core.api.ThrowingConsumer; import org.h2.jdbcx.JdbcDataSource; import org.hsqldb.jdbc.JDBCDriver; -import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.DisplayName; import org.junit.jupiter.api.Test; @@ -128,32 +126,19 @@ static void setUp() { prepareConnectionPoolDatasources(); } - @AfterAll - static void tearDown() { - cpDatasources - .values() - .forEach( - k -> - k.values() - .forEach( - dataSource -> { - if (dataSource instanceof Closeable) { - try { - ((Closeable) dataSource).close(); - } catch (IOException ignored) { - // ignore exceptions during close - } - } - })); - } - static void prepareConnectionPoolDatasources() { List connectionPoolNames = asList("tomcat", "hikari", "c3p0"); connectionPoolNames.forEach( cpName -> { Map dbDsMapping = new HashMap<>(); jdbcUrls.forEach( - (dbType, jdbcUrl) -> dbDsMapping.put(dbType, createDs(cpName, dbType, jdbcUrl))); + (dbType, jdbcUrl) -> { + DataSource dataSource = createDs(cpName, dbType, jdbcUrl); + if (dataSource instanceof Closeable) { + cleanup.deferAfterAll((Closeable) dataSource); + } + dbDsMapping.put(dbType, dataSource); + }); cpDatasources.put(cpName, dbDsMapping); }); } @@ -1172,7 +1157,7 @@ void testConnectionConstructorThrowing( try { connection = new TestConnection(true); - } catch (Exception ignored) { + } catch (IllegalStateException ignored) { connection = driver.connect(jdbcUrl, null); } connection = wrap(connection); @@ -1584,7 +1569,7 @@ void testHandleRecursiveStatements( // https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6015 @DisplayName("test proxy statement") @Test - void testProxyStatement() throws Exception { + void testProxyStatement() throws ClassNotFoundException, SQLException { Connection connection = wrap(new org.h2.Driver().connect(jdbcUrls.get("h2"), null)); cleanup.deferCleanup(connection); Statement statement = connection.createStatement(); diff --git a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/ProxyStatementFactory.java b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/ProxyStatementFactory.java index 743f1eb54f83..6b03d73e7bf8 100644 --- a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/ProxyStatementFactory.java +++ b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/ProxyStatementFactory.java @@ -14,7 +14,8 @@ class ProxyStatementFactory { - static Statement proxyStatementWithCustomClassLoader(Statement statement) throws Exception { + static Statement proxyStatementWithCustomClassLoader(Statement statement) + throws ClassNotFoundException { TestClassLoader classLoader = new TestClassLoader(ProxyStatementFactory.class.getClassLoader()); Class testInterface = classLoader.loadClass(TestInterface.class.getName()); if (testInterface.getClassLoader() != classLoader) { From 5b51d035f4c360cbba85cb50900979cbb9dece50 Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 05:23:25 +0000 Subject: [PATCH 2/8] Review fixes for jedis-1.4:javaagent Automated code review of instrumentation/jedis/jedis-1.4/javaagent. --- .../javaagent/instrumentation/jedis/v1_4/JedisRequest.java | 2 +- .../javaagent/instrumentation/jedis/v1_4/JedisSingletons.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java index f40d3301ed82..e416cbbde3b1 100644 --- a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java +++ b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java @@ -13,7 +13,7 @@ import redis.clients.jedis.Protocol; @AutoValue -public abstract class JedisRequest { +abstract class JedisRequest { public static JedisRequest create(Connection connection, Protocol.Command command) { return new AutoValue_JedisRequest(connection, command, emptyList()); diff --git a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java index 2ddb38437d9c..525ba95635e5 100644 --- a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java +++ b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java @@ -13,7 +13,7 @@ import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter; import io.opentelemetry.instrumentation.api.instrumenter.SpanKindExtractor; -public class JedisSingletons { +class JedisSingletons { private static final String INSTRUMENTATION_NAME = "io.opentelemetry.jedis-1.4"; private static final Instrumenter instrumenter; From dcabd3341b49b88146b63b30a909846e026889ee Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 05:48:14 +0000 Subject: [PATCH 3/8] Review fixes for jedis-1.4:testing Automated code review of instrumentation/jedis/jedis-1.4/testing. --- .../instrumentation/jedis/AbstractJedisTest.java | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/instrumentation/jedis/jedis-1.4/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/AbstractJedisTest.java b/instrumentation/jedis/jedis-1.4/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/AbstractJedisTest.java index ac6199d2d915..f25a838291e9 100644 --- a/instrumentation/jedis/jedis-1.4/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/AbstractJedisTest.java +++ b/instrumentation/jedis/jedis-1.4/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/AbstractJedisTest.java @@ -21,10 +21,9 @@ import static org.assertj.core.api.Assertions.assertThat; import io.opentelemetry.api.trace.SpanKind; +import io.opentelemetry.instrumentation.testing.internal.AutoCleanupExtension; import io.opentelemetry.instrumentation.testing.junit.AgentInstrumentationExtension; import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension; -import java.io.IOException; -import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; @@ -37,6 +36,9 @@ public abstract class AbstractJedisTest { @RegisterExtension private static final InstrumentationExtension testing = AgentInstrumentationExtension.create(); + @RegisterExtension + private static final AutoCleanupExtension cleanup = AutoCleanupExtension.create(); + private static final GenericContainer redisServer = new GenericContainer<>("redis:6.2.3-alpine").withExposedPorts(6379); @@ -49,15 +51,11 @@ public abstract class AbstractJedisTest { @BeforeAll static void setup() { redisServer.start(); + cleanup.deferAfterAll(redisServer::stop); host = redisServer.getHost(); port = redisServer.getMappedPort(6379); jedis = new Jedis(host, port); - } - - @AfterAll - static void cleanup() throws IOException { - jedis.disconnect(); - redisServer.stop(); + cleanup.deferAfterAll(jedis::disconnect); } @BeforeEach From e08dec9ebee8b73839e0386fc25e353bbc6df193 Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 06:03:26 +0000 Subject: [PATCH 4/8] Review fixes for jedis-3.0:javaagent Automated code review of instrumentation/jedis/jedis-3.0/javaagent. --- .../instrumentation/jedis/v3_0/JedisRequest.java | 2 +- .../instrumentation/jedis/v3_0/JedisSingletons.java | 2 +- .../jedis/v3_0/Jedis30ClientTest.java | 13 ++++++------- 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java index 5c38108d134a..68a4120ce704 100644 --- a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java +++ b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java @@ -17,7 +17,7 @@ import redis.clients.jedis.commands.ProtocolCommand; @AutoValue -public abstract class JedisRequest { +abstract class JedisRequest { private static final RedisCommandSanitizer sanitizer = RedisCommandSanitizer.create( diff --git a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java index 9eecfbdcbbfe..2a2377ecf257 100644 --- a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java +++ b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java @@ -13,7 +13,7 @@ import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter; import io.opentelemetry.instrumentation.api.instrumenter.SpanKindExtractor; -public class JedisSingletons { +class JedisSingletons { private static final String INSTRUMENTATION_NAME = "io.opentelemetry.jedis-3.0"; private static final Instrumenter instrumenter; diff --git a/instrumentation/jedis/jedis-3.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/Jedis30ClientTest.java b/instrumentation/jedis/jedis-3.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/Jedis30ClientTest.java index 509c6b4c03d5..dbae5ceb7a7e 100644 --- a/instrumentation/jedis/jedis-3.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/Jedis30ClientTest.java +++ b/instrumentation/jedis/jedis-3.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/Jedis30ClientTest.java @@ -27,12 +27,12 @@ import static org.assertj.core.api.Assertions.assertThat; import io.opentelemetry.api.trace.SpanKind; +import io.opentelemetry.instrumentation.testing.internal.AutoCleanupExtension; import io.opentelemetry.instrumentation.testing.junit.AgentInstrumentationExtension; import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension; import java.net.InetAddress; import java.net.UnknownHostException; import org.assertj.core.api.AbstractLongAssert; -import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; @@ -45,6 +45,9 @@ class Jedis30ClientTest { @RegisterExtension private static final InstrumentationExtension testing = AgentInstrumentationExtension.create(); + @RegisterExtension + private static final AutoCleanupExtension cleanup = AutoCleanupExtension.create(); + private static final GenericContainer redisServer = new GenericContainer<>("redis:6.2.3-alpine").withExposedPorts(6379); @@ -59,16 +62,12 @@ class Jedis30ClientTest { @BeforeAll static void setup() throws UnknownHostException { redisServer.start(); + cleanup.deferAfterAll(redisServer::stop); host = redisServer.getHost(); ip = InetAddress.getByName(host).getHostAddress(); port = redisServer.getMappedPort(6379); jedis = new Jedis(host, port); - } - - @AfterAll - static void cleanup() { - redisServer.stop(); - jedis.close(); + cleanup.deferAfterAll(jedis); } @BeforeEach From 151070fa79fdcb247347948a485521eb5485b0c2 Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 06:07:00 +0000 Subject: [PATCH 5/8] Review fixes for jedis-4.0:javaagent Automated code review of instrumentation/jedis/jedis-4.0/javaagent. --- .../jedis/v4_0/Jedis40ClientTest.java | 23 +++++++++---------- 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/instrumentation/jedis/jedis-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v4_0/Jedis40ClientTest.java b/instrumentation/jedis/jedis-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v4_0/Jedis40ClientTest.java index 2a4afa37dfb8..213b1ef603bd 100644 --- a/instrumentation/jedis/jedis-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v4_0/Jedis40ClientTest.java +++ b/instrumentation/jedis/jedis-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jedis/v4_0/Jedis40ClientTest.java @@ -22,11 +22,11 @@ import static org.assertj.core.api.Assertions.assertThat; import io.opentelemetry.api.trace.SpanKind; +import io.opentelemetry.instrumentation.testing.internal.AutoCleanupExtension; import io.opentelemetry.instrumentation.testing.junit.AgentInstrumentationExtension; import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension; import java.net.InetAddress; import java.net.UnknownHostException; -import org.junit.jupiter.api.AfterAll; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; @@ -37,29 +37,28 @@ @SuppressWarnings("deprecation") // using deprecated semconv class Jedis40ClientTest { @RegisterExtension - static final InstrumentationExtension testing = AgentInstrumentationExtension.create(); + private static final InstrumentationExtension testing = AgentInstrumentationExtension.create(); - static GenericContainer redisServer = + @RegisterExtension + private static final AutoCleanupExtension cleanup = AutoCleanupExtension.create(); + + private static final GenericContainer redisServer = new GenericContainer<>("redis:6.2.3-alpine").withExposedPorts(6379); - static String ip; + private static String ip; - static int port; + private static int port; - static Jedis jedis; + private static Jedis jedis; @BeforeAll static void setup() throws UnknownHostException { redisServer.start(); + cleanup.deferAfterAll(redisServer::stop); port = redisServer.getMappedPort(6379); ip = InetAddress.getByName(redisServer.getHost()).getHostAddress(); jedis = new Jedis(redisServer.getHost(), port); - } - - @AfterAll - static void cleanup() { - redisServer.stop(); - jedis.close(); + cleanup.deferAfterAll(jedis); } @BeforeEach From fb1fcb14295872a1899ad4dbc3fa8864834e27c7 Mon Sep 17 00:00:00 2001 From: otelbot <197425009+otelbot@users.noreply.github.com> Date: Mon, 27 Apr 2026 06:29:27 +0000 Subject: [PATCH 6/8] Review fixes for jetty-httpclient-12.0:javaagent Automated code review of instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent. --- ...JettyClient12ResponseListenersInstrumentation.java | 11 ++--------- .../v12_0/JettyHttpClient12Instrumentation.java | 2 +- 2 files changed, 3 insertions(+), 10 deletions(-) diff --git a/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyClient12ResponseListenersInstrumentation.java b/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyClient12ResponseListenersInstrumentation.java index 9672475e03fa..0e64a9d6b2f5 100644 --- a/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyClient12ResponseListenersInstrumentation.java +++ b/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyClient12ResponseListenersInstrumentation.java @@ -59,11 +59,7 @@ public static Scope onEnterNotify(@Advice.Argument(0) Response response) { } @Advice.OnMethodExit(suppress = Throwable.class, onThrowable = Throwable.class, inline = false) - public static void onExitNotify( - @Advice.Argument(0) Response response, - @Advice.Thrown Throwable throwable, - @Advice.Enter @Nullable Scope scope) { - + public static void onExitNotify(@Advice.Enter @Nullable Scope scope) { if (scope != null) { scope.close(); } @@ -82,10 +78,7 @@ public static Scope onEnterComplete(@Advice.Argument(0) Result result) { } @Advice.OnMethodExit(suppress = Throwable.class, onThrowable = Throwable.class, inline = false) - public static void onExitComplete( - @Advice.Argument(0) Result result, - @Advice.Thrown Throwable throwable, - @Advice.Enter @Nullable Scope scope) { + public static void onExitComplete(@Advice.Enter @Nullable Scope scope) { if (scope != null) { scope.close(); } diff --git a/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyHttpClient12Instrumentation.java b/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyHttpClient12Instrumentation.java index c6ad785e4326..d46ab888c929 100644 --- a/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyHttpClient12Instrumentation.java +++ b/instrumentation/jetty-httpclient/jetty-httpclient-12.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jetty/httpclient/v12_0/JettyHttpClient12Instrumentation.java @@ -63,7 +63,7 @@ public static AdviceLocals onEnterSend(@Advice.This HttpRequest request) { if (context == null) { return null; } - // set context for responseListeners + // store the parent context for request/response listener callbacks request.attribute(JETTY_CLIENT_CONTEXT_KEY, parentContext); return new AdviceLocals(context, context.makeCurrent()); From 46508f08d527bb6c693d09deb80f045e8f1f1233 Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Mon, 27 Apr 2026 08:09:22 -0700 Subject: [PATCH 7/8] Restore public visibility for advice-referenced jedis classes JedisRequest and JedisSingletons are referenced from advice woven into redis.clients.jedis.Connection; reducing them to package-private caused IllegalAccessError at runtime. --- .../javaagent/instrumentation/jedis/v1_4/JedisRequest.java | 2 +- .../javaagent/instrumentation/jedis/v1_4/JedisSingletons.java | 2 +- .../javaagent/instrumentation/jedis/v3_0/JedisRequest.java | 2 +- .../javaagent/instrumentation/jedis/v3_0/JedisSingletons.java | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java index e416cbbde3b1..f40d3301ed82 100644 --- a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java +++ b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisRequest.java @@ -13,7 +13,7 @@ import redis.clients.jedis.Protocol; @AutoValue -abstract class JedisRequest { +public abstract class JedisRequest { public static JedisRequest create(Connection connection, Protocol.Command command) { return new AutoValue_JedisRequest(connection, command, emptyList()); diff --git a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java index 525ba95635e5..2ddb38437d9c 100644 --- a/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java +++ b/instrumentation/jedis/jedis-1.4/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v1_4/JedisSingletons.java @@ -13,7 +13,7 @@ import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter; import io.opentelemetry.instrumentation.api.instrumenter.SpanKindExtractor; -class JedisSingletons { +public class JedisSingletons { private static final String INSTRUMENTATION_NAME = "io.opentelemetry.jedis-1.4"; private static final Instrumenter instrumenter; diff --git a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java index 68a4120ce704..5c38108d134a 100644 --- a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java +++ b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisRequest.java @@ -17,7 +17,7 @@ import redis.clients.jedis.commands.ProtocolCommand; @AutoValue -abstract class JedisRequest { +public abstract class JedisRequest { private static final RedisCommandSanitizer sanitizer = RedisCommandSanitizer.create( diff --git a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java index 2a2377ecf257..9eecfbdcbbfe 100644 --- a/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java +++ b/instrumentation/jedis/jedis-3.0/javaagent/src/main/java/io/opentelemetry/javaagent/instrumentation/jedis/v3_0/JedisSingletons.java @@ -13,7 +13,7 @@ import io.opentelemetry.instrumentation.api.instrumenter.Instrumenter; import io.opentelemetry.instrumentation.api.instrumenter.SpanKindExtractor; -class JedisSingletons { +public class JedisSingletons { private static final String INSTRUMENTATION_NAME = "io.opentelemetry.jedis-3.0"; private static final Instrumenter instrumenter; From 79389f014aeccccf169e7d899679529f9396e90c Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Mon, 27 Apr 2026 09:13:24 -0700 Subject: [PATCH 8/8] revert --- .../jdbc/testing/AbstractJdbcInstrumentationTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java index 08ec89d84e84..686443701260 100644 --- a/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java +++ b/instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/testing/AbstractJdbcInstrumentationTest.java @@ -1157,7 +1157,7 @@ void testConnectionConstructorThrowing( try { connection = new TestConnection(true); - } catch (IllegalStateException ignored) { + } catch (Exception ignored) { connection = driver.connect(jdbcUrl, null); } connection = wrap(connection); @@ -1569,7 +1569,7 @@ void testHandleRecursiveStatements( // https://github.com/open-telemetry/opentelemetry-java-instrumentation/issues/6015 @DisplayName("test proxy statement") @Test - void testProxyStatement() throws ClassNotFoundException, SQLException { + void testProxyStatement() throws Exception { Connection connection = wrap(new org.h2.Driver().connect(jdbcUrls.get("h2"), null)); cleanup.deferCleanup(connection); Statement statement = connection.createStatement();