diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/EventSourceInitializer.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/EventSourceInitializer.java
index 45f33037d4..79b15380a5 100644
--- a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/EventSourceInitializer.java
+++ b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/EventSourceInitializer.java
@@ -18,6 +18,7 @@ public interface EventSourceInitializer<P extends HasMetadata> {
    * 
    * @param context a {@link EventSourceContext} providing access to information useful to event
    *        sources
+   * @return list of event sources to register
    */
   List<EventSource> prepareEventSources(EventSourceContext<P> context);
 
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/ReconciliationMaxInterval.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/ReconciliationMaxInterval.java
index 05459e7123..b2c1f1c255 100644
--- a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/ReconciliationMaxInterval.java
+++ b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/ReconciliationMaxInterval.java
@@ -15,13 +15,13 @@
    * reconciliation is scheduled with a target interval after the last reconciliation. Note that
    * this not applies for retries, in case of an exception reconciliation is not scheduled. This is
    * not a fixed rate, in other words a new reconciliation is scheduled after each reconciliation.
-   * <p/>
+   * <p>
    * If an interval is specified by {@link UpdateControl} or {@link DeleteControl}, those take
    * precedence.
-   * <p/>
+   * <p>
    * This is a fail-safe feature, in the sense that if informers are in place and the reconciler
    * implementation is correct, this feature can be turned off.
-   * <p/>
+   * <p>
    * Use NO_RECONCILIATION_MAX_INTERVAL in {@link Constants} to turn off this feature.
    *
    * @return max delay between reconciliations
diff --git a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/UpdateControl.java b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/UpdateControl.java
index c388958a8a..1762198286 100644
--- a/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/UpdateControl.java
+++ b/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/api/reconciler/UpdateControl.java
@@ -23,6 +23,10 @@ private UpdateControl(
    * Creates an update control instance that instructs the framework to do an update on resource
    * itself, not on the status. Note that usually as a results of a reconciliation should be a
    * status update not an update to the resource itself.
+   *
+   * @param <T> custom resource type
+   * @param customResource customResource to use for update
+   * @return initialized update control
    */
   public static <T extends HasMetadata> UpdateControl<T> updateResource(T customResource) {
     return new UpdateControl<>(customResource, false, true);