Review comments

Author: zsmoore

src/main/java/org/dataloader/DataLoaderRegistry.java

@@ -368,11 +368,25 @@ public Builder registerAll(DataLoaderRegistry otherRegistry) {
             return this;
         }
 
+        /**
+         * The {@link DataLoaderInstrumentation} to use for this registry
+         *
+         * @param instrumentation instrumentation to use
+         *
+         * @return the builder for a fluent pattern
+         */
         public Builder instrumentation(DataLoaderInstrumentation instrumentation) {
             this.instrumentation = instrumentation;
             return this;
         }
 
+        /**
+         * The {@link DispatchStrategy} to use for this registry
+         *
+         * @param dispatchStrategy strategy to use
+         *
+         * @return this builder for a fluent pattern
+         */
         public Builder dispatchStrategy(DispatchStrategy dispatchStrategy) {
             this.dispatchStrategy = dispatchStrategy;
             return this;

src/main/java/org/dataloader/DispatchStrategy.java

@@ -5,21 +5,37 @@
 
 import java.util.concurrent.atomic.AtomicBoolean;
 
+/**
+ * An interface to implement to allow for custom dispatch strategies when executing {@link DataLoader}s
+ */
 @NullMarked
 @PublicApi
 public interface DispatchStrategy {
 
+    /**
+     * A {@link DispatchStrategy} that does nothing
+     */
     DispatchStrategy NO_OP = new DispatchStrategy() {
     };
 
+    /**
+     * Lifecycle method called when the registry is created that this dispatch strategy is attached to
+     * @param registry the {@link DataLoaderRegistry} this dispatch strategy is attached to
+     */
     default void onRegistryCreation(DataLoaderRegistry registry) {
 
     }
 
+    /**
+     * Called when a {@link DataLoader#load(Object)} is called on a dataloader
+     */
     default void loadCalled() {
 
     }
 
+    /**
+     * Called when a {@link DataLoader#load(Object)} is executed and completed on a dataloader
+     */
     default void loadCompleted() {
 
     }