Suppressing some errorprone checks with flags

Author: schcamille

google-cloud-spanner-executor/src/main/java/com/google/cloud/executor/spanner/CloudClientExecutor.java

@@ -2862,8 +2862,6 @@ private com.google.spanner.v1.StructType buildStructType(StructReader struct) {
   }
 
   /** Convert a struct to a proto(value list) for constructing result rows and struct values. */
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings({"ProtoBuilderReturnValueIgnored", "ReturnValueIgnored"})
   private com.google.spanner.executor.v1.ValueList buildStruct(StructReader struct) {
     com.google.spanner.executor.v1.ValueList.Builder structBuilder =
         com.google.spanner.executor.v1.ValueList.newBuilder();

google-cloud-spanner/src/test/java/com/google/cloud/spanner/AbstractReadContextTest.java

@@ -266,8 +266,7 @@ public void testGetExecuteBatchDmlRequestBuilderWithPriority() {
     assertEquals(Priority.PRIORITY_LOW, request.getRequestOptions().getPriority());
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("JUnit4TestNotRun")
+  @Test
   public void executeSqlRequestBuilderWithRequestOptions() {
     ExecuteSqlRequest request =
         context

google-cloud-spanner/src/test/java/com/google/cloud/spanner/BackupIdTest.java

@@ -40,8 +40,6 @@ public void basics() {
     assertThat(bid.toString()).isEqualTo(name);
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void badName() {
     IllegalArgumentException e =

google-cloud-spanner/src/test/java/com/google/cloud/spanner/ChannelUsageTest.java

@@ -228,8 +228,6 @@ public void testCreatesNumChannels() {
     assertEquals(numChannels, batchCreateSessionLocalIps.size());
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void testUsesAllChannels() throws InterruptedException {
     final int multiplier = 2;

google-cloud-spanner/src/test/java/com/google/cloud/spanner/DatabaseAdminClientTest.java

@@ -632,8 +632,6 @@ public void instanceListDatabaseOperations()
     assertThat(instance.listDatabaseOperations().iterateAll()).hasSize(6);
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void instanceListDatabaseOperationsWithMetadata() throws Exception {
     Instance instance =
@@ -716,8 +714,6 @@ public void instanceListBackupOperations()
     assertThat(instance.listBackupOperations().iterateAll()).hasSize(2);
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void instanceListBackupOperationsWithProgress() throws InvalidProtocolBufferException {
     Instance instance =

google-cloud-spanner/src/test/java/com/google/cloud/spanner/OpenTelemetryBuiltInMetricsTracerTest.java

@@ -196,8 +196,6 @@ public void testMetricsSingleUseQuery() {
     assertThat(getAggregatedValue(attemptCountMetricData, expectedAttributes)).isEqualTo(1);
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void testMetricsWithGaxRetryUnaryRpc() {
     Stopwatch stopwatch = Stopwatch.createStarted();

google-cloud-spanner/src/test/java/com/google/cloud/spanner/RandomResultSetGenerator.java

@@ -78,8 +78,6 @@ public class RandomResultSetGenerator {
             .build(),
       };
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings({"ProtoBuilderReturnValueIgnored", "ReturnValueIgnored"})
   private static ResultSetMetadata generateMetadata() {
     StructType.Builder rowTypeBuilder = StructType.newBuilder();
     for (int col = 0; col < TYPES.length; col++) {

google-cloud-spanner/src/test/java/com/google/cloud/spanner/SelectRandomBenchmark.java

@@ -113,8 +113,6 @@ public void teardown() throws Exception {
   }
 
   /** Measures the time needed to execute a burst of read requests. */
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("ReturnValueIgnored")
   @Benchmark
   public void burstRead(final BenchmarkState server) throws Exception {
     int totalQueries = server.maxSessions * 8;

google-cloud-spanner/src/test/java/com/google/cloud/spanner/TypeTest.java

@@ -501,8 +501,6 @@ public void emptyStruct() {
     assertProtoEquals(t.toProto(), "code: STRUCT struct_type {}");
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void structFieldIndexNotFound() {
     Type t = Type.struct(StructField.of("f1", Type.int64()));
@@ -511,8 +509,6 @@ public void structFieldIndexNotFound() {
     assertThat(e.getMessage().contains("Field not found: f2"));
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings("CheckReturnValue")
   @Test
   public void structFieldIndexAmbiguous() {
     Type t = Type.struct(StructField.of("f1", Type.int64()), StructField.of("f1", Type.string()));

google-cloud-spanner/src/test/java/com/google/cloud/spanner/connection/RandomResultSetGenerator.java

@@ -179,8 +179,6 @@ private static void appendProtoTypes(List<Type> types, Dialect dialect) {
     }
   }
 
-  // Suppressed for initial Error Prone rollout.
-  @SuppressWarnings({"ProtoBuilderReturnValueIgnored", "ReturnValueIgnored"})
   public static ResultSetMetadata generateAllTypesMetadata(Type[] types) {
     StructType.Builder rowTypeBuilder = StructType.newBuilder();
     for (int col = 0; col < types.length; col++) {

pom.xml

@@ -151,7 +151,8 @@
               <fork>true</fork>
               <compilerArgs>
                 <arg>-XDcompilePolicy=simple</arg>
-                <arg>-Xplugin:ErrorProne</arg>
+                <arg>-Xplugin:ErrorProne -Xep:CheckReturnValue:OFF \
+                -Xep:ProtoBuilderReturnValueIgnored:OFF -Xep:ReturnValueIgnored:OFF</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED</arg>
                 <arg>-J--add-exports=jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED</arg>