[ErrorProne] Fix InvalidParam, InvalidInlineTag, InvalidBlockTag, and InvalidLink checks (#37773)

* Fix InvalidInlineTag, InvalidParam, InvalidBlockTag and InvalidLink javadocs

* Fix JdbcUtil after merge

* spotless

* changes

* leave ignore block

* Fix InvalidLink and restore InvalidBlockTag to disabledChecks

* Remove duplicate entry

Author: stankiewicz

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

@@ -1545,9 +1545,6 @@ class BeamModulePlugin implements Plugin<Project> {
             "ExtendsAutoValue",
             "InlineMeSuggester",
             "InvalidBlockTag",
-            "InvalidInlineTag",
-            "InvalidLink",
-            "InvalidParam",
             "InvalidThrows",
             "JavaTimeDefaultTimeZone",
             "JavaUtilDate",

examples/java/webapis/src/main/java/org/apache/beam/examples/webapis/ImageRequest.java

@@ -54,7 +54,7 @@ static Builder builder() {
     return new AutoValue_ImageRequest.Builder();
   }
 
-  /** Build an {@link ImageRequest} from a {@param url}. */
+  /** Build an {@link ImageRequest} from a {@code url}. */
   static ImageRequest of(String url) {
     return builder().setImageUrl(url).setMimeType(mimeTypeOf(url)).build();
   }

it/google-cloud-platform/src/main/java/org/apache/beam/it/gcp/LoadTestBase.java

@@ -211,7 +211,7 @@ protected boolean waitForNumMessages(String jobId, String pcollection, Long expe
    *
    * @param metrics a map of raw metrics. The results are also appened in the map.
    * @param launchInfo Job info of the job
-   * @param config a {@class MetricsConfiguration}
+   * @param config a {@link MetricsConfiguration}
    */
   private void computeDataflowMetrics(
       Map<String, Double> metrics, LaunchInfo launchInfo, MetricsConfiguration config)
@@ -365,7 +365,7 @@ protected Map<String, Double> getCpuUtilizationMetrics(String jobId, TimeInterva
    * Computes throughput metrics of the given pcollection in dataflow job.
    *
    * @param jobInfo dataflow job LaunchInfo
-   * @param config the {@class MetricsConfiguration}
+   * @param config the {@link MetricsConfiguration}
    * @param timeInterval interval for the monitoring query
    * @return throughput metrics of the pcollection
    */

it/google-cloud-platform/src/test/java/org/apache/beam/it/gcp/storage/FileBasedIOLT.java

@@ -251,7 +251,7 @@ static class Configuration extends SyntheticSourceOptions {
     /** Number of dynamic destinations to write. */
     @JsonProperty public int numShards = 0;
 
-    /** See {@class org.apache.beam.sdk.io.Compression}. */
+    /** See {@link org.apache.beam.sdk.io.Compression}. */
     @JsonProperty public String compressionType = "UNCOMPRESSED";
 
     /** Runner specified to run the pipeline. */

runners/core-java/src/main/java/org/apache/beam/runners/core/StatefulDoFnRunner.java

@@ -49,8 +49,7 @@
 /**
  * A customized {@link DoFnRunner} that handles late data dropping and garbage collection for
  * stateful {@link DoFn DoFns}. It registers a GC timer in {@link #processElement(WindowedValue)}
- * and does cleanup in {@link #onTimer(String, String, BoundedWindow, Instant, Instant, TimeDomain,
- * boolean)}
+ * and does cleanup in {@link #onTimer}
  *
  * @param <InputT> the type of the {@link DoFn} (main) input elements
  * @param <OutputT> the type of the {@link DoFn} (main) output elements

runners/direct-java/src/main/java/org/apache/beam/runners/direct/DirectTimerInternals.java

@@ -95,14 +95,14 @@ public void deleteTimer(
             timeDomain));
   }
 
-  /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+  /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
   @Deprecated
   @Override
   public void deleteTimer(StateNamespace namespace, String timerId, String timerFamilyId) {
     throw new UnsupportedOperationException("Canceling of timer by ID is not yet supported.");
   }
 
-  /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+  /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
   @Deprecated
   @Override
   public void deleteTimer(TimerData timerData) {

runners/flink/1.19/src/test/java/org/apache/beam/runners/flink/streaming/StreamSources.java

@@ -52,7 +52,7 @@ public interface OutputWrapper<T> extends Output<T> {
     @Override
     default void emitWatermarkStatus(WatermarkStatus watermarkStatus) {}
 
-    /** In Flink 1.19 the {@code emitRecordAttributes} method was added. */
+    /** In Flink 1.19 the {@code recordAttributes} method was added. */
     @Override
     default void emitRecordAttributes(RecordAttributes recordAttributes) {
       throw new UnsupportedOperationException("emitRecordAttributes not implemented");

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

@@ -1655,7 +1655,7 @@ void onFiredOrDeletedTimer(TimerData timer) {
       }
     }
 
-    /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+    /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
     @Deprecated
     @Override
     public void deleteTimer(StateNamespace namespace, String timerId, String timerFamilyId) {
@@ -1672,7 +1672,7 @@ public void deleteTimer(
       }
     }
 
-    /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+    /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
     @Override
     @Deprecated
     public void deleteTimer(TimerData timer) {

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

@@ -1655,7 +1655,7 @@ void onFiredOrDeletedTimer(TimerData timer) {
       }
     }
 
-    /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+    /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
     @Deprecated
     @Override
     public void deleteTimer(StateNamespace namespace, String timerId, String timerFamilyId) {
@@ -1672,7 +1672,7 @@ public void deleteTimer(
       }
     }
 
-    /** @deprecated use {@link #deleteTimer(StateNamespace, String, TimeDomain)}. */
+    /** @deprecated use {@link #deleteTimer(StateNamespace, String, String, TimeDomain)}. */
     @Override
     @Deprecated
     public void deleteTimer(TimerData timer) {

runners/flink/2.0/src/test/java/org/apache/beam/runners/flink/streaming/StreamSources.java

@@ -52,7 +52,7 @@ public interface OutputWrapper<T> extends Output<T> {
     @Override
     default void emitWatermarkStatus(WatermarkStatus watermarkStatus) {}
 
-    /** In Flink 1.19 the {@code emitRecordAttributes} method was added. */
+    /** In Flink 1.19 the {@code recordAttributes} method was added. */
     @Override
     default void emitRecordAttributes(RecordAttributes recordAttributes) {
       throw new UnsupportedOperationException("emitRecordAttributes not implemented");