(beam) branch master updated: [ErrorProne] Enable UnsafeReflectiveConstructionCast check (#37784)

[yhu]

This is an automated email from the ASF dual-hosted git repository.

yhu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 3b472aaa9d3 [ErrorProne] Enable UnsafeReflectiveConstructionCast check 
(#37784)
3b472aaa9d3 is described below

commit 3b472aaa9d3f4788b612d57f23abe43534c3c186
Author: Radosław Stankiewicz <radosl...@google.com>
AuthorDate: Sun Mar 15 02:02:33 2026 +0100

    [ErrorProne] Enable UnsafeReflectiveConstructionCast check (#37784)
---
 .../org/apache/beam/gradle/BeamModulePlugin.groovy     |  1 -
 .../extensions/avro/schemas/utils/AvroUtilsTest.java   | 18 +++++++++---------
 2 files changed, 9 insertions(+), 10 deletions(-)

diff --git 
a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy 
b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
index a6627cda2db..2705752e26b 100644
--- a/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
+++ b/buildSrc/src/main/groovy/org/apache/beam/gradle/BeamModulePlugin.groovy
@@ -1561,7 +1561,6 @@ class BeamModulePlugin implements Plugin<Project> {
             "UnnecessaryMethodReference",
             "UnnecessaryParentheses",
             "UnrecognisedJavadocTag",
-            "UnsafeReflectiveConstructionCast",
             // errorprone 3.2.0+ checks
             "DirectInvocationOnMock",
             "Finalize",
diff --git 
a/sdks/java/extensions/avro/src/test/java/org/apache/beam/sdk/extensions/avro/schemas/utils/AvroUtilsTest.java
 
b/sdks/java/extensions/avro/src/test/java/org/apache/beam/sdk/extensions/avro/schemas/utils/AvroUtilsTest.java
index 9e0519ae4eb..57da8e77bb1 100644
--- 
a/sdks/java/extensions/avro/src/test/java/org/apache/beam/sdk/extensions/avro/schemas/utils/AvroUtilsTest.java
+++ 
b/sdks/java/extensions/avro/src/test/java/org/apache/beam/sdk/extensions/avro/schemas/utils/AvroUtilsTest.java
@@ -92,17 +92,17 @@ public class AvroUtilsTest {
     Iterable<?> data;
     if (VERSION_AVRO.equals("1.8.2")) {
       data =
-          (Iterable<?>)
-              Class.forName("org.apache.avro.RandomData")
-                  .getDeclaredConstructor(org.apache.avro.Schema.class, 
Integer.TYPE)
-                  .newInstance(schema, maxLength);
+          Class.forName("org.apache.avro.RandomData")
+              .asSubclass(Iterable.class)
+              .getDeclaredConstructor(org.apache.avro.Schema.class, 
Integer.TYPE)
+              .newInstance(schema, maxLength);
     } else {
       data =
-          (Iterable<?>)
-              Class.forName("org.apache.avro.util.RandomData")
-                  .getDeclaredConstructor(org.apache.avro.Schema.class, 
Integer.TYPE, Boolean.TYPE)
-                  // force Utf8 in random data to match with String type used 
in AvroUtils
-                  .newInstance(schema, maxLength, true);
+          Class.forName("org.apache.avro.util.RandomData")
+              .asSubclass(Iterable.class)
+              .getDeclaredConstructor(org.apache.avro.Schema.class, 
Integer.TYPE, Boolean.TYPE)
+              // force Utf8 in random data to match with String type used in 
AvroUtils
+              .newInstance(schema, maxLength, true);
     }
     return data;
   }