Skip to content

Commit de321b5

Browse files
committed
TEZ-4669: Spotbugs fails in precommit - SQUASHED together with fake patch, edit before merging!!!!
1 parent 1fc0035 commit de321b5

3 files changed

Lines changed: 8 additions & 0 deletions

File tree

pom.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1012,6 +1012,12 @@
10121012
<artifactId>spotbugs</artifactId>
10131013
<version>${spotbugs.version}</version>
10141014
</dependency>
1015+
<!-- Upgraded plexus-velocity is required by spotbugs since TEZ-4669 -->
1016+
<dependency>
1017+
<groupId>org.codehaus.plexus</groupId>
1018+
<artifactId>plexus-velocity</artifactId>
1019+
<version>2.3.0</version>
1020+
</dependency>
10151021
</dependencies>
10161022
<configuration>
10171023
<excludeFilterFile>${basedir}/findbugs-exclude.xml</excludeFilterFile>

tez-api/pom.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
</parent>
2525
<artifactId>tez-api</artifactId>
2626

27+
2728
<properties>
2829
<maven.javadoc.skip>false</maven.javadoc.skip>
2930
</properties>

tez-api/src/test/java/org/apache/tez/common/TestTezCommonUtils.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -60,6 +60,7 @@ public class TestTezCommonUtils {
6060

6161
@BeforeClass
6262
public static void setup() throws Exception {
63+
LOG.info("HEY, THIS IS FOR SPOTBUGS, DON'T MERGE");
6364
conf.set(TezConfiguration.TEZ_AM_STAGING_DIR, STAGE_DIR);
6465
LOG.info("Starting mini clusters");
6566
try {

0 commit comments

Comments
 (0)