Skip to content

Commit c46c6af

Browse files
zhtttylzzhtttylz
committed
HADOOP-19731. Fix SpotBugs warnings introduced after SpotBugs version upgrade.
1 parent dd8fd7e commit c46c6af

File tree

28 files changed

+113
-39
lines changed

28 files changed

+113
-39
lines changed

‎dev-support/findbugs-exclude-global.xml‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
<?xml version="1.0" encoding="UTF-8"?>
2+
<FindBugsFilter>
3+
<Match>
4+
<Bug category="EXPERIMENTAL"/>
5+
</Match>
6+
<Match>
7+
<Bug code="AT"/>
8+
</Match>
9+
<Match>
10+
<Bug code="CT"/>
11+
</Match>
12+
<Match>
13+
<Bug pattern="PA_PUBLIC_PRIMITIVE_ATTRIBUTE"/>
14+
</Match>
15+
<Match>
16+
<Bug pattern="NN_NAKED_NOTIFY"/>
17+
</Match>
18+
</FindBugsFilter>

‎hadoop-cloud-storage-project/hadoop-cos/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,10 @@
6969
<artifactId>spotbugs-maven-plugin</artifactId>
7070
<configuration>
7171
<xmlOutput>true</xmlOutput>
72-
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
73-
</excludeFilterFile>
72+
<excludeFilterFiles combine.children="append">
73+
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
74+
</excludeFilterFile>
75+
</excludeFilterFiles>
7476
<effort>Max</effort>
7577
</configuration>
7678
</plugin>

‎hadoop-cloud-storage-project/hadoop-huaweicloud/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,10 @@
6464
<artifactId>spotbugs-maven-plugin</artifactId>
6565
<configuration>
6666
<xmlOutput>true</xmlOutput>
67-
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
68-
</excludeFilterFile>
67+
<excludeFilterFiles combine.children="append">
68+
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
69+
</excludeFilterFile>
70+
</excludeFilterFiles>
6971
<effort>Max</effort>
7072
</configuration>
7173
</plugin>

‎hadoop-cloud-storage-project/hadoop-tos/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,10 @@
184184
<artifactId>spotbugs-maven-plugin</artifactId>
185185
<configuration>
186186
<xmlOutput>true</xmlOutput>
187-
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
188-
</excludeFilterFile>
187+
<excludeFilterFiles combine.children="append">
188+
<excludeFilterFile>${basedir}/dev-support/findbugs-exclude.xml
189+
</excludeFilterFile>
190+
</excludeFilterFiles>
189191
<effort>Max</effort>
190192
</configuration>
191193
</plugin>

‎hadoop-common-project/hadoop-auth/pom.xml‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,9 @@
267267
<groupId>com.github.spotbugs</groupId>
268268
<artifactId>spotbugs-maven-plugin</artifactId>
269269
<configuration>
270-
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml</excludeFilterFile>
270+
<excludeFilterFiles combine.children="append">
271+
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml</excludeFilterFile>
272+
</excludeFilterFiles>
271273
</configuration>
272274
</plugin>
273275
</plugins>

‎hadoop-common-project/hadoop-kms/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -244,8 +244,10 @@
244244
<groupId>com.github.spotbugs</groupId>
245245
<artifactId>spotbugs-maven-plugin</artifactId>
246246
<configuration>
247-
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
248-
</excludeFilterFile>
247+
<excludeFilterFiles combine.children="append">
248+
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
249+
</excludeFilterFile>
250+
</excludeFilterFiles>
249251
</configuration>
250252
</plugin>
251253
</plugins>

‎hadoop-common-project/hadoop-minikdc/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,8 +76,10 @@
7676
<groupId>com.github.spotbugs</groupId>
7777
<artifactId>spotbugs-maven-plugin</artifactId>
7878
<configuration>
79-
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
80-
</excludeFilterFile>
79+
<excludeFilterFiles combine.children="append">
80+
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
81+
</excludeFilterFile>
82+
</excludeFilterFiles>
8183
</configuration>
8284
</plugin>
8385
</plugins>

‎hadoop-common-project/hadoop-nfs/pom.xml‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,8 +125,10 @@
125125
<groupId>com.github.spotbugs</groupId>
126126
<artifactId>spotbugs-maven-plugin</artifactId>
127127
<configuration>
128-
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
129-
</excludeFilterFile>
128+
<excludeFilterFiles combine.children="append">
129+
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml
130+
</excludeFilterFile>
131+
</excludeFilterFiles>
130132
</configuration>
131133
</plugin>
132134
</plugins>

‎hadoop-common-project/hadoop-registry/pom.xml‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,10 @@
193193
<artifactId>spotbugs-maven-plugin</artifactId>
194194
<configuration>
195195
<xmlOutput>true</xmlOutput>
196-
<excludeFilterFile>${project.basedir}/dev-support/findbugs-exclude.xml</excludeFilterFile>
196+
<excludeFilterFiles combine.children="append">
197+
<excludeFilterFile>${project.basedir}/dev-support/findbugs-exclude.xml
198+
</excludeFilterFile>
199+
</excludeFilterFiles>
197200
<effort>Max</effort>
198201
</configuration>
199202
</plugin>

‎hadoop-hdfs-project/hadoop-hdfs-httpfs/pom.xml‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -365,7 +365,9 @@
365365
<groupId>com.github.spotbugs</groupId>
366366
<artifactId>spotbugs-maven-plugin</artifactId>
367367
<configuration>
368-
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml</excludeFilterFile>
368+
<excludeFilterFiles combine.children="append">
369+
<excludeFilterFile>${basedir}/dev-support/findbugsExcludeFile.xml</excludeFilterFile>
370+
</excludeFilterFiles>
369371
</configuration>
370372
</plugin>
371373
</plugins>

0 commit comments

Comments
 (0)