Fixing oversight of missing synchronized on long version of recalibrate

dougqh · dougqh · commit 890133bd7221 · 2025-09-05T13:48:30.000-04:00
diff --git a/dd-trace-core/src/main/java/datadog/trace/common/writer/ddagent/GenerationalUtf8Cache.java b/dd-trace-core/src/main/java/datadog/trace/common/writer/ddagent/GenerationalUtf8Cache.java
@@ -156,7 +156,7 @@ public synchronized void recalibrate() {
    *
    * <p>While still racy this method is synchronized to avoid simultaneous recalibrations
    */
-  public void recalibrate(long accessTimeMs) {
+  public synchronized void recalibrate(long accessTimeMs) {
     this.accessTimeMs = accessTimeMs;
 
     recalibrate(this.edenEntries);

Original file line number	Diff line number	Diff line change
`@@ -156,7 +156,7 @@ public synchronized void recalibrate() {`
`156`	`156`	`*`
`157`	`157`	`* <p>While still racy this method is synchronized to avoid simultaneous recalibrations`
`158`	`158`	`*/`
`159`		`- public void recalibrate(long accessTimeMs) {`
	`159`	`+ public synchronized void recalibrate(long accessTimeMs) {`
`160`	`160`	`this.accessTimeMs = accessTimeMs;`
`161`	`161`
`162`	`162`	`recalibrate(this.edenEntries);`