[incubator-uniffle] branch master updated: [SpotBugs] Enable SC_START_IN_CTOR check (#541)

[ckj]

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

ckj pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new e2096d61 [SpotBugs] Enable SC_START_IN_CTOR check (#541)
e2096d61 is described below

commit e2096d6126ec66853ed49e2ba74cde178bd7e404
Author: Kaijie Chen <c...@apache.org>
AuthorDate: Thu Feb 2 17:21:51 2023 +0800

    [SpotBugs] Enable SC_START_IN_CTOR check (#541)
    
    ### What changes were proposed in this pull request?
    
    Remove SC_START_IN_CTOR from `spotbugs-exclude.xml`.
    It was excluded in #517 by mistake.
    
    ### Why are the changes needed?
    
    #462
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    CI.
---
 spotbugs-exclude.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/spotbugs-exclude.xml b/spotbugs-exclude.xml
index cae8bb7e..01e57ee3 100644
--- a/spotbugs-exclude.xml
+++ b/spotbugs-exclude.xml
@@ -20,6 +20,6 @@
     <Package name="~org\.apache\.uniffle\.proto.*"/>
   </Match>
   <Match>
-    <Bug 
pattern="MS_EXPOSE_REP,EI_EXPOSE_REP,EI_EXPOSE_REP2,EI_EXPOSE_STATIC_REP2,THROWS_METHOD_THROWS_RUNTIMEEXCEPTION,THROWS_METHOD_THROWS_CLAUSE_BASIC_EXCEPTION,THROWS_METHOD_THROWS_CLAUSE_THROWABLE,MS_PKGPROTECT,MS_CANNOT_BE_FINAL,SE_BAD_FIELD,SC_START_IN_CTOR,SWL_SLEEP_WITH_LOCK_HELD,IS2_INCONSISTENT_SYNC"
 />
+    <Bug 
pattern="MS_EXPOSE_REP,EI_EXPOSE_REP,EI_EXPOSE_REP2,EI_EXPOSE_STATIC_REP2,THROWS_METHOD_THROWS_RUNTIMEEXCEPTION,THROWS_METHOD_THROWS_CLAUSE_BASIC_EXCEPTION,THROWS_METHOD_THROWS_CLAUSE_THROWABLE,MS_PKGPROTECT,MS_CANNOT_BE_FINAL,SE_BAD_FIELD,SWL_SLEEP_WITH_LOCK_HELD,IS2_INCONSISTENT_SYNC"
 />
   </Match>
 </FindBugsFilter>