fix: do not create a new RateLimiter every time by default

metacosm · metacosm · commit ceba879af67e · 2022-07-04T11:59:00.000+02:00
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ControllerConfiguration.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/config/ControllerConfiguration.java
@@ -17,6 +17,8 @@
 
 public interface ControllerConfiguration<R extends HasMetadata> extends ResourceConfiguration<R> {
 
+  RateLimiter DEFAULT_RATE_LIMITER = new PeriodRateLimiter();
+
   default String getName() {
     return ReconcilerUtils.getDefaultReconcilerName(getAssociatedReconcilerClassName());
   }
@@ -46,7 +48,7 @@ default RetryConfiguration getRetryConfiguration() {
   }
 
   default RateLimiter getRateLimiter() {
-    return new PeriodRateLimiter();
+    return DEFAULT_RATE_LIMITER;
   }
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,8 @@`
`17`	`17`
`18`	`18`	`public interface ControllerConfiguration<R extends HasMetadata> extends ResourceConfiguration<R> {`
`19`	`19`
	`20`	`+ RateLimiter DEFAULT_RATE_LIMITER = new PeriodRateLimiter();`
	`21`	`+`
`20`	`22`	`default String getName() {`
`21`	`23`	`return ReconcilerUtils.getDefaultReconcilerName(getAssociatedReconcilerClassName());`
`22`	`24`	`}`
`@@ -46,7 +48,7 @@ default RetryConfiguration getRetryConfiguration() {`
`46`	`48`	`}`
`47`	`49`
`48`	`50`	`default RateLimiter getRateLimiter() {`
`49`		`- return new PeriodRateLimiter();`
	`51`	`+ return DEFAULT_RATE_LIMITER;`
`50`	`52`	`}`
`51`	`53`
`52`	`54`	`/**`