Skip to content

Commit 2d8eef9

Browse files
authored
Review fixes for runtime-telemetry-java17:library (#17665)
Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com>
1 parent c87f8db commit 2d8eef9

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

instrumentation/runtime-telemetry/runtime-telemetry-java17/library/src/main/java/io/opentelemetry/instrumentation/runtimemetrics/java17/RuntimeMetrics.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ public final class RuntimeMetrics implements AutoCloseable {
3333
* @param openTelemetry the {@link OpenTelemetry} instance used to record telemetry
3434
*/
3535
public static RuntimeMetrics create(OpenTelemetry openTelemetry) {
36-
return new RuntimeMetricsBuilder(openTelemetry).build();
36+
return builder(openTelemetry).build();
3737
}
3838

3939
/**

instrumentation/runtime-telemetry/runtime-telemetry-java17/library/src/test/java/io/opentelemetry/instrumentation/runtimemetrics/java17/RuntimeMetricsBuilderTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ class RuntimeMetricsBuilderTest {
2828
static void setup() {
2929
try {
3030
Class.forName("jdk.jfr.FlightRecorder");
31-
} catch (ClassNotFoundException exception) {
31+
} catch (ClassNotFoundException ignored) {
3232
Assumptions.abort("JFR not present");
3333
}
3434
Assumptions.assumeTrue(FlightRecorder.isAvailable(), "JFR not available");

instrumentation/runtime-telemetry/runtime-telemetry-java17/library/src/test/java/io/opentelemetry/instrumentation/runtimemetrics/java17/RuntimeMetricsTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ class RuntimeMetricsTest {
3636
void setup() {
3737
try {
3838
Class.forName("jdk.jfr.FlightRecorder");
39-
} catch (ClassNotFoundException exception) {
39+
} catch (ClassNotFoundException ignored) {
4040
Assumptions.abort("JFR not present");
4141
}
4242
Assumptions.assumeTrue(FlightRecorder.isAvailable(), "JFR not available");

0 commit comments

Comments
 (0)