-
Task
-
Resolution: Unresolved
-
Minor
-
None
-
None
-
None
-
False
-
-
False
-
-
The KafkaRebalance Operator is inherently complex, and as a result, the KafkaRebalanceAssemblyOperator code has become increasingly intricate over time. We've already had to apply suppression annotations to manage Checkstyle warnings—such as @SuppressWarnings({"checkstyle:NPathComplexity"}) on the main reconcile method, and @SuppressWarnings({"checkstyle:ClassFanOutComplexity", "checkstyle:CyclomaticComplexity"}) on the class itself.
We should refactor this class in a way that follows the pattern of the other reconcilers [2] [3] in the cluster-operator module that breaks the nested asynchronous calls into smaller methods and the reconciled state between those methods. This will make it easier to fix bugs, add features, and add more test coverage.