fix: make it clear that ConfiguredController.init should not be called

metacosm · metacosm · commit 9012076d7bc1 · 2021-09-24T13:06:10.000+02:00
Fixes #554
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/ConfiguredController.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/ConfiguredController.java
@@ -105,8 +105,7 @@ public UpdateControl<R> execute() {
 
   @Override
   public void init(EventSourceManager eventSourceManager) {
-    this.manager = eventSourceManager;
-    controller.init(eventSourceManager);
+    throw new UnsupportedOperationException("This method should never be called directly");
   }
 
   @Override

Original file line number	Diff line number	Diff line change
`@@ -105,8 +105,7 @@ public UpdateControl<R> execute() {`
`105`	`105`
`106`	`106`	`@Override`
`107`	`107`	`public void init(EventSourceManager eventSourceManager) {`
`108`		`- this.manager = eventSourceManager;`
`109`		`- controller.init(eventSourceManager);`
	`108`	`+ throw new UnsupportedOperationException("This method should never be called directly");`
`110`	`109`	`}`
`111`	`110`
`112`	`111`	`@Override`