Fixed findingBugs

Author: chiacyu

hadoop-hdds/client/dev-support/findbugsExcludeFile.xml

@@ -19,4 +19,16 @@
     <Class name="org.apache.hadoop.hdds.scm.storage.ByteArrayReader"></Class>
     <Bug pattern="EI_EXPOSE_REP2" /> <!-- "Deep copy byte[] has bad impact on performance" -->
   </Match>
+  <Match>
+    <Class name="org.apache.hadoop.hdds.scm.storage.DummyBlockInputStreamWithRetry"/>
+    <Bug pattern="RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT" />
+  </Match>
+  <Match>
+    <Class name="org.apache.hadoop.hdds.scm.storage.TestBlockInputStream"/>
+    <Bug pattern="RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT" />
+  </Match>
+  <Match>
+    <Class name="org.apache.hadoop.ozone.client.io.TestBlockInputStreamFactoryImpl"/>
+    <Bug pattern="RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT" />
+  </Match>
 </FindBugsFilter>

hadoop-hdds/client/src/test/java/org/apache/hadoop/hdds/scm/storage/DummyBlockInputStreamWithRetry.java

@@ -40,7 +40,6 @@
  * A dummy BlockInputStream with pipeline refresh function to mock read
  * block call to DN.
  */
-@SuppressWarnings("java:S2201")
 final class DummyBlockInputStreamWithRetry
     extends DummyBlockInputStream {
 

hadoop-hdds/client/src/test/java/org/apache/hadoop/hdds/scm/storage/TestBlockInputStream.java

@@ -293,7 +293,6 @@ public void testRefreshPipelineFunction() throws Exception {
 
   @ParameterizedTest
   @MethodSource("exceptionsTriggersRefresh")
-  @SuppressWarnings("java:S2201")
   void refreshesPipelineOnReadFailure(IOException ex) throws Exception {
     // GIVEN
     Pipeline pipeline = MockPipeline.createSingleNodePipeline();
@@ -407,7 +406,6 @@ public void testReadNotRetriedOnOtherException(IOException ex)
 
   @ParameterizedTest
   @MethodSource("exceptionsTriggersRefresh")
-  @SuppressWarnings("java:S2201")
   public void testRefreshOnReadFailureAfterUnbuffer(IOException ex)
       throws Exception {
     // GIVEN

hadoop-hdds/client/src/test/java/org/apache/hadoop/ozone/client/io/TestBlockInputStreamFactoryImpl.java

@@ -51,7 +51,6 @@ public class TestBlockInputStreamFactoryImpl {
   private OzoneConfiguration conf = new OzoneConfiguration();
 
   @Test
-  @SuppressWarnings("java:S2201")
   public void testNonECGivesBlockInputStream() throws IOException {
     BlockInputStreamFactory factory = new BlockInputStreamFactoryImpl();
     ReplicationConfig repConfig =