Update PR: Add a test method to verify that only one log entry is allowed per minute.

devilcoolyue · devilcoolyue · commit f0bf98512d9b · 2025-03-02T11:06:27.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,8 @@ 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,8 @@ 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(`
	`45`	`+ THROTTLED_RATE_LIMIT / rateTimeUnit.toSeconds(1), THROTTLED_RATE_LIMIT, clock);`
`45`	`46`	`}`
`46`	`47`
`47`	`48`	`/** Log a message at the given level. */`