From 0baf06bad49fb1d4a9b1d6b1682a4cd52ec393e3 Mon Sep 17 00:00:00 2001 From: Bruce Bujon Date: Tue, 3 Dec 2024 12:18:32 +0100 Subject: [PATCH] feat(bootstrap): Refactor telemetry point capture --- .../BootstrapInitializationTelemetry.java | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/dd-java-agent/src/main/java/datadog/trace/bootstrap/BootstrapInitializationTelemetry.java b/dd-java-agent/src/main/java/datadog/trace/bootstrap/BootstrapInitializationTelemetry.java index 2827790f90bf..c896983af227 100644 --- a/dd-java-agent/src/main/java/datadog/trace/bootstrap/BootstrapInitializationTelemetry.java +++ b/dd-java-agent/src/main/java/datadog/trace/bootstrap/BootstrapInitializationTelemetry.java @@ -109,19 +109,13 @@ public void initMetaInfo(String attr, String value) { @Override public void onAbort(String reasonCode) { - synchronized (this.points) { - this.points.add("library_entrypoint.abort"); - this.points.add("reason:" + reasonCode); - } + onPoint("library_entrypoint.abort", "reason:" + reasonCode); markIncomplete(); } @Override public void onError(Throwable t) { - synchronized (this.points) { - this.points.add("library_entrypoint.error"); - this.points.add("error_type:" + t.getClass().getName()); - } + onPoint("library_entrypoint.error", "error_type:" + t.getClass().getName()); } @Override @@ -132,9 +126,13 @@ public void onFatalError(Throwable t) { @Override public void onError(String reasonCode) { + onPoint("library_entrypoint.error", "error_type:" + reasonCode); + } + + private void onPoint(String name, String tag) { synchronized (this.points) { - this.points.add("library_entrypoint.error"); - this.points.add("error_type:" + reasonCode); + this.points.add(name); + this.points.add(tag); } }