respond to PR comments

Author: Quinn-With-Two-Ns

temporal-sdk/src/main/java/io/temporal/internal/replay/ReplayWorkflowContext.java

@@ -251,6 +251,7 @@ void sideEffect(
    *
    * @param id id of the side effect call. It links multiple calls together. Calls with different
    *     ids are completely independent.
+   * @param userMetadata user metadata to attach to the marker event.
    * @param func function that gets as input a result of a previous {@code mutableSideEffect} call.
    *     The function executes its business logic (like checking config value) and if value didn't
    *     change returns {@link Optional#empty()}. If value has changed and needs to be recorded in

temporal-sdk/src/main/java/io/temporal/internal/statemachines/SideEffectStateMachine.java

@@ -74,6 +74,7 @@ enum State {
   /**
    * Creates new SideEffect Marker
    *
+   * @param metadata user metadata to attach to the side effect marker.
    * @param func used to produce side effect value. null if replaying.
    * @param callback returns side effect value or failure
    * @param commandSink callback to send commands to