fix: Correct malformed Javadoc tags and update Error Prone configuration. (#37755)

Author: stankiewicz

buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy

@@ -1553,7 +1553,6 @@ class BeamModulePlugin implements Plugin<Project> {
             "JavaTimeDefaultTimeZone",
             "JavaUtilDate",
             "JodaConstructors",
-            "MalformedInlineTag",
             "MixedMutabilityReturnType",
             "PreferJavaTimeOverload",
             "NonCanonicalType",

runners/flink/1.20/src/main/java/org/apache/beam/runners/flink/translation/wrappers/streaming/DoFnOperator.java

@@ -596,9 +596,9 @@ DoFnRunner<InputT, OutputT> createBufferingDoFnRunnerIfNeeded(
   }
 
   /**
-   * Retrieve a keyed state backend that should be used to buffer elements for {@link @{code @}
-   * RequiresStableInput} functionality. By default this is the default keyed backend, but can be
-   * override in @{link ExecutableStageDoFnOperator}.
+   * Retrieve a keyed state backend that should be used to buffer elements for
+   * {@code @RequiresStableInput} functionality. By default this is the default keyed backend, but
+   * can be override in {@link ExecutableStageDoFnOperator}.
    *
    * @return the keyed backend to use for element buffering
    */

runners/flink/2.0/src/main/java/org/apache/beam/runners/flink/translation/wrappers/streaming/DoFnOperator.java

@@ -596,9 +596,9 @@ DoFnRunner<InputT, OutputT> createBufferingDoFnRunnerIfNeeded(
   }
 
   /**
-   * Retrieve a keyed state backend that should be used to buffer elements for {@link @{code @}
-   * RequiresStableInput} functionality. By default this is the default keyed backend, but can be
-   * override in @{link ExecutableStageDoFnOperator}.
+   * Retrieve a keyed state backend that should be used to buffer elements for
+   * {@code @RequiresStableInput} functionality. By default this is the default keyed backend, but
+   * can be override in {@link ExecutableStageDoFnOperator}.
    *
    * @return the keyed backend to use for element buffering
    */

runners/flink/src/main/java/org/apache/beam/runners/flink/translation/wrappers/streaming/DoFnOperator.java

@@ -598,9 +598,9 @@ DoFnRunner<InputT, OutputT> createBufferingDoFnRunnerIfNeeded(
   }
 
   /**
-   * Retrieve a keyed state backend that should be used to buffer elements for {@link @{code @}
-   * RequiresStableInput} functionality. By default this is the default keyed backend, but can be
-   * override in @{link ExecutableStageDoFnOperator}.
+   * Retrieve a keyed state backend that should be used to buffer elements for
+   * {@code @RequiresStableInput} functionality. By default this is the default keyed backend, but
+   * can be override in {@link ExecutableStageDoFnOperator}.
    *
    * @return the keyed backend to use for element buffering
    */

sdks/java/extensions/euphoria/src/main/java/org/apache/beam/sdk/extensions/euphoria/core/client/operator/ReduceByKey.java

@@ -240,7 +240,7 @@ default WindowByBuilder<KeyT, ValueT> combineBy(CombineFunctionWithIdentity<Valu
      * Syntactic sugar to enable #combineBy to take only single argument and be used in helpers like
      * #combineBy(Sums.ofLongs()).
      *
-     * @deprecated Replaced by @{link #combineBy(CombineFunctionWithIdentity)}.
+     * @deprecated Replaced by {@link #combineBy(CombineFunctionWithIdentity)}.
      */
     @Deprecated
     default WindowByBuilder<KeyT, ValueT> combineBy(

sdks/java/extensions/euphoria/src/main/java/org/apache/beam/sdk/extensions/euphoria/core/client/operator/ReduceWindow.java

@@ -192,7 +192,7 @@ default WindowByBuilder<ValueT> combineBy(
      * Syntactic sugar to enable #combineBy to take only single argument and be used in helpers like
      * #combineBy(Sums.ofLongs()).
      *
-     * @deprecated Replaced by @{link #combineBy(ReduceByKey.CombineFunctionWithIdentity)}.
+     * @deprecated Replaced by {@link #combineBy(ReduceByKey.CombineFunctionWithIdentity)}.
      */
     @Deprecated
     default WindowByBuilder<ValueT> combineBy(

sdks/java/extensions/euphoria/src/test/java/org/apache/beam/sdk/extensions/euphoria/core/client/util/SumsTest.java

@@ -25,7 +25,7 @@
 import org.junit.runner.RunWith;
 import org.junit.runners.JUnit4;
 
-/** Test suite for @{link Sums}. */
+/** Test suite for {@link Sums}. */
 @RunWith(JUnit4.class)
 public class SumsTest {
 

sdks/java/io/rrio/src/main/java/org/apache/beam/io/requestresponse/SetupTeardown.java

@@ -20,7 +20,7 @@
 import java.io.Serializable;
 
 /**
- * Provided by user and called within {@link org.apache.beam.sdk.transforms.DoFn.Setup} and @{link
+ * Provided by user and called within {@link org.apache.beam.sdk.transforms.DoFn.Setup} and {@link
  * org.apache.beam.sdk.transforms.DoFn.Teardown} lifecycle methods of {@link Call}'s {@link
  * org.apache.beam.sdk.transforms.DoFn}.
  */

sdks/java/io/synthetic/src/main/java/org/apache/beam/sdk/io/synthetic/BundleSplitter.java

@@ -24,7 +24,7 @@
 import java.util.List;
 import org.apache.beam.sdk.io.range.OffsetRange;
 
-/** Determines bundle sizes based on @{link SyntheticSourceOptions}. */
+/** Determines bundle sizes based on {@link SyntheticSourceOptions}. */
 public class BundleSplitter implements Serializable {
 
   private SyntheticSourceOptions options;