fix bug: throttledRateLimiter is not once per minute, but five times

devilcoolyue · devilcoolyue · commit 4fe326fe162f · 2025-02-27T17:53:34.000+08:00
diff --git a/sdk/common/src/main/java/io/opentelemetry/sdk/internal/ThrottlingLogger.java b/sdk/common/src/main/java/io/opentelemetry/sdk/internal/ThrottlingLogger.java
@@ -41,7 +41,7 @@ public ThrottlingLogger(Logger delegate) {
     this.fastRateLimiter =
         new RateLimiter(RATE_LIMIT / rateTimeUnit.toSeconds(1), RATE_LIMIT, clock);
     this.throttledRateLimiter =
-        new RateLimiter(RATE_LIMIT / rateTimeUnit.toSeconds(1), THROTTLED_RATE_LIMIT, clock);
+        new RateLimiter(THROTTLED_RATE_LIMIT / rateTimeUnit.toSeconds(1), THROTTLED_RATE_LIMIT, clock);
   }
 
   /** Log a message at the given level. */

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public ThrottlingLogger(Logger delegate) {`
`41`	`41`	`this.fastRateLimiter =`
`42`	`42`	`new RateLimiter(RATE_LIMIT / rateTimeUnit.toSeconds(1), RATE_LIMIT, clock);`
`43`	`43`	`this.throttledRateLimiter =`
`44`		`- new RateLimiter(RATE_LIMIT / rateTimeUnit.toSeconds(1), THROTTLED_RATE_LIMIT, clock);`
	`44`	`+ new RateLimiter(THROTTLED_RATE_LIMIT / rateTimeUnit.toSeconds(1), THROTTLED_RATE_LIMIT, clock);`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`/** Log a message at the given level. */`