Reverting unwanted changes in spotbugs-exclude.xml & checkstyle-suppressions.xml (Azure#25362)

Co-authored-by: Aigerim Beishenbekova <aigerimb@DESKTOP-D4IJIGC.redmond.corp.microsoft.com>

Author: AikoBB

eng/code-quality-reports/src/main/resources/checkstyle/checkstyle-suppressions.xml

@@ -44,7 +44,7 @@
 
   <!-- Fields tenantId, clientId and clientSecret are not set in all constructors. -->
   <suppress checks="com.azure.tools.checkstyle.checks.EnforceFinalFieldsCheck" files="com.azure.security.keyvault.jca.implementation.KeyVaultClient"/>
-
+  
   <!-- Fields defaultTrustManager and keyStore might be set more than once. -->
   <suppress checks="com.azure.tools.checkstyle.checks.EnforceFinalFieldsCheck" files="com.azure.security.keyvault.jca.KeyVaultTrustManager"/>
 
@@ -510,6 +510,7 @@ the main ServiceBusClientBuilder. -->
   <suppress checks="MethodName" files="com.azure.spring.data.gremlin.common.GremlinConfig.java"/>
 
   <suppress checks="MethodName" files="com.azure.spring.integration.core.api.CheckpointConfig.java"/>
+  <suppress checks="MethodName" files="com.azure.spring.integration.core.api.BatchConsumerConfig.java"/>
   <suppress checks="MethodName" files="com.azure.spring.integration.servicebus.ServiceBusClientConfig.java"/>
 
   <!-- Checkstyle suppressions for azure.spring.data.cosmos package -->

eng/code-quality-reports/src/main/resources/spotbugs/spotbugs-exclude.xml

@@ -1310,6 +1310,13 @@
     <Bug pattern="REC_CATCH_EXCEPTION"/>
   </Match>
 
+    <!-- Exception is required to handle when getting java version  -->
+  <Match>
+    <Class name="com.azure.cosmos.implementation.Utils"/>
+    <Method name="getJavaVersion"/>
+    <Bug pattern="REC_CATCH_EXCEPTION"/>
+  </Match>
+
   <!-- Exception is required to handle and fallback to default pair  -->
   <Match>
     <Class name="com.azure.cosmos.implementation.ResourceId"/>
@@ -1883,6 +1890,13 @@
     <Bug pattern="DE_MIGHT_IGNORE"/>
   </Match>
 
+  <!-- Exception in AutoCloseable.close is ignored -->
+  <Match>
+    <Class name="com.azure.core.tracing.opentelemetry.OpenTelemetryHttpPolicy$ScalarPropagatingMono"/>
+    <Method name="subscribe"/>
+    <Bug pattern="DE_MIGHT_IGNORE"/>
+  </Match>
+
   <!-- Exception is ignored by design which indicate that non-parsable id -->
   <Match>
     <Class name="com.azure.cosmos.implementation.ResourceId"/>
@@ -2414,6 +2428,10 @@
     <Method name="~(get|post)"/>
     <Bug pattern="RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE"/>
   </Match>
+  <Match>
+    <Class name="com.azure.security.keyvault.jca.implementation.KeyVaultPrivateKey"/>
+    <Bug pattern="SE_BAD_FIELD"/>
+  </Match>
 
   <!-- Returning a new copy of the object is not necessary for queuing system use cases.
   Client do not change content of message once message is created. Array copy would degrade the performance.
@@ -2499,4 +2517,13 @@
     <Method name="getServiceVersions"/>
     <Bug pattern="BC_UNCONFIRMED_CAST_OF_RETURN_VALUE"/>
   </Match>
+
+  <Match>
+    <Class name="~com\.azure\.search.*"/>
+    <Or>
+      <Field name="odataType"/>
+      <Field name="type"/>
+    </Or>
+    <Bug pattern="URF_UNREAD_FIELD"/>
+  </Match>
 </FindBugsFilter>