Repository: hbase Updated Branches: refs/heads/branch-1.1 8b3385bd3 -> ca5e1a577
HBASE-16321 ensure no findbugs-jsr305 Signed-off-by: Esteban Gutierrez <[email protected]> Conflicts: hbase-client/src/main/java/org/apache/hadoop/hbase/MetaTableAccessor.java hbase-client/src/main/java/org/apache/hadoop/hbase/client/ConnectionImplementation.java hbase-rsgroup/pom.xml hbase-server/pom.xml hbase-server/src/main/java/org/apache/hadoop/hbase/coprocessor/ObserverContext.java hbase-server/src/main/java/org/apache/hadoop/hbase/master/TableStateManager.java hbase-server/src/main/java/org/apache/hadoop/hbase/util/FSTableDescriptors.java hbase-server/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java hbase-spark/pom.xml src/main/asciidoc/_chapters/developer.adoc Conflicts: hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java hbase-server/src/test/java/org/apache/hadoop/hbase/mapreduce/TestMultiTableSnapshotInputFormat.java Ammended: hbase-client/src/main/java/org/apache/hadoop/hbase/MetaTableAccessor.java Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/ca5e1a57 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ca5e1a57 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ca5e1a57 Branch: refs/heads/branch-1.1 Commit: ca5e1a57712166928c5dd6670580fd82f7918f4d Parents: 8b3385b Author: Sean Busbey <[email protected]> Authored: Tue Aug 2 01:17:59 2016 -0500 Committer: Sean Busbey <[email protected]> Committed: Fri Aug 5 16:30:58 2016 -0500 ---------------------------------------------------------------------- hbase-client/pom.xml | 8 +- .../hbase/protobuf/ResponseConverter.java | 1 + .../hbase/zookeeper/MetaTableLocator.java | 2 - .../apache/hadoop/hbase/codec/BaseDecoder.java | 5 +- hbase-server/pom.xml | 5 ++ .../hadoop/hbase/master/LoadBalancer.java | 1 + .../hbase/client/TestMetaWithReplicas.java | 2 - pom.xml | 87 ++++++++++++++------ 8 files changed, 73 insertions(+), 38 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-client/pom.xml ---------------------------------------------------------------------- diff --git a/hbase-client/pom.xml b/hbase-client/pom.xml index 74364ba..37e3e4e 100644 --- a/hbase-client/pom.xml +++ b/hbase-client/pom.xml @@ -242,8 +242,8 @@ </activation> <dependencies> <dependency> - <groupId>com.google.code.findbugs</groupId> - <artifactId>jsr305</artifactId> + <groupId>com.github.stephenc.findbugs</groupId> + <artifactId>findbugs-annotations</artifactId> <optional>true</optional> </dependency> <dependency> @@ -255,10 +255,6 @@ <artifactId>hadoop-common</artifactId> <exclusions> <exclusion> - <groupId>com.github.stephenc.findbugs</groupId> - <artifactId>findbugs-annotations</artifactId> - </exclusion> - <exclusion> <groupId>net.java.dev.jets3t</groupId> <artifactId>jets3t</artifactId> </exclusion> http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java ---------------------------------------------------------------------- diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java index 14db92b..4bc5f6e 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ResponseConverter.java @@ -21,6 +21,7 @@ import java.io.IOException; import java.util.ArrayList; import java.util.List; +import edu.umd.cs.findbugs.annotations.Nullable; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.hadoop.hbase.classification.InterfaceAudience; http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MetaTableLocator.java ---------------------------------------------------------------------- diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MetaTableLocator.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MetaTableLocator.java index b76c83e..e8815fd 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MetaTableLocator.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MetaTableLocator.java @@ -63,8 +63,6 @@ import java.net.UnknownHostException; import java.util.List; import java.util.ArrayList; -import javax.annotation.Nullable; - /** * Utility class to perform operation (get/wait for/verify/set/delete) on znode in ZooKeeper * which keeps hbase:meta region server location. http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-common/src/main/java/org/apache/hadoop/hbase/codec/BaseDecoder.java ---------------------------------------------------------------------- diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/codec/BaseDecoder.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/codec/BaseDecoder.java index 86f8678..485b88a 100644 --- a/hbase-common/src/main/java/org/apache/hadoop/hbase/codec/BaseDecoder.java +++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/codec/BaseDecoder.java @@ -22,8 +22,7 @@ import java.io.IOException; import java.io.InputStream; import java.io.PushbackInputStream; -import javax.annotation.Nonnull; - +import edu.umd.cs.findbugs.annotations.NonNull; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.apache.hadoop.hbase.Cell; @@ -99,7 +98,7 @@ public abstract class BaseDecoder implements Codec.Decoder { * thrown if EOF is reached prematurely. Does not return null. * @throws IOException */ - @Nonnull + @NonNull protected abstract Cell parseCell() throws IOException; @Override http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-server/pom.xml ---------------------------------------------------------------------- diff --git a/hbase-server/pom.xml b/hbase-server/pom.xml index 4424d27..e1692cf 100644 --- a/hbase-server/pom.xml +++ b/hbase-server/pom.xml @@ -431,6 +431,11 @@ </dependency> <!-- General dependencies --> <dependency> + <groupId>com.github.stephenc.findbugs</groupId> + <artifactId>findbugs-annotations</artifactId> + <optional>true</optional> + </dependency> + <dependency> <groupId>com.yammer.metrics</groupId> <artifactId>metrics-core</artifactId> </dependency> http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java index f979403..33bc9c1 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/LoadBalancer.java @@ -21,6 +21,7 @@ package org.apache.hadoop.hbase.master; import java.util.List; import java.util.Map; +import edu.umd.cs.findbugs.annotations.Nullable; import org.apache.hadoop.hbase.classification.InterfaceAudience; import org.apache.hadoop.hbase.conf.ConfigurationObserver; import org.apache.hadoop.conf.Configurable; http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java ---------------------------------------------------------------------- diff --git a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java index 354dc66..4907d3d 100644 --- a/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java +++ b/hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java @@ -18,8 +18,6 @@ */ package org.apache.hadoop.hbase.client; -import javax.annotation.Nullable; - import java.io.IOException; import static org.apache.hadoop.hbase.util.hbck.HbckTestingUtil.assertErrors; import static org.apache.hadoop.hbase.util.hbck.HbckTestingUtil.doFsck; http://git-wip-us.apache.org/repos/asf/hbase/blob/ca5e1a57/pom.xml ---------------------------------------------------------------------- diff --git a/pom.xml b/pom.xml index aa617e8..1fe39ed 100644 --- a/pom.xml +++ b/pom.xml @@ -814,36 +814,52 @@ </dependency> </dependencies> <!-- version set by parent --> - <configuration> - <rules> - <!-- The earliest maven version we verify builds for via ASF Jenkins --> - <requireMavenVersion> - <version>[${maven.min.version},)</version> - <message>Maven is out of date. + <executions> + <execution> + <id>min-maven-and-java</id> + <goals> + <goal>enforce</goal> + </goals> + <configuration> + <rules> + <!-- The earliest maven version we verify builds for via ASF Jenkins --> + <requireMavenVersion> + <version>[${maven.min.version},)</version> + <message>Maven is out of date. HBase requires at least version ${maven.min.version} of Maven to properly build from source. You appear to be using an older version. You can use either "mvn -version" or "mvn enforcer:display-info" to verify what version is active. See the reference guide on building for more information: http://hbase.apache.org/book.html#build - </message> - </requireMavenVersion> - <!-- The earliest JVM version we verify builds for via ASF Jenkins --> - <requireJavaVersion> - <version>[${java.min.version},)</version> - <message>Java is out of date. + </message> + </requireMavenVersion> + <!-- The earliest JVM version we verify builds for via ASF Jenkins --> + <requireJavaVersion> + <version>[${java.min.version},)</version> + <message>Java is out of date. HBase requirs at least version ${java.min.version} of the JDK to properly build from source. You appear to be using an older version. You can use either "mvn -version" or "mvn enforcer:display-info" to verify what version is active. See the reference guide on building for more information: http://hbase.apache.org/book.html#build - </message> - </requireJavaVersion> - </rules> - </configuration> - <executions> + </message> + </requireJavaVersion> + </rules> + </configuration> + </execution> <execution> - <id>enforce</id> + <id>banned-jsr305</id> <goals> <goal>enforce</goal> </goals> + <configuration> + <rules> + <bannedDependencies> + <excludes> + <exclude>com.google.code.findbugs:jsr305</exclude> + </excludes> + <message>We don't allow the JSR305 jar from the Findbugs project, see HBASE-16321.</message> + </bannedDependencies> + </rules> + </configuration> </execution> </executions> </plugin> @@ -1164,7 +1180,6 @@ <httpclient.version>3.1</httpclient.version> <metrics-core.version>2.2.0</metrics-core.version> <guava.version>12.0.1</guava.version> - <jsr305.version>1.3.9</jsr305.version> <jackson.version>1.9.13</jackson.version> <jasper.version>5.5.23</jasper.version> <jaxb-api.version>2.2.2</jaxb-api.version> @@ -1402,6 +1417,12 @@ </dependency> <!-- General dependencies --> <dependency> + <groupId>com.github.stephenc.findbugs</groupId> + <artifactId>findbugs-annotations</artifactId> + <version>${findbugs-annotations}</version> + </dependency> + <!-- General dependencies --> + <dependency> <groupId>org.codehaus.jettison</groupId> <artifactId>jettison</artifactId> <version>${jettison.version}</version> @@ -1427,11 +1448,12 @@ <groupId>com.google.guava</groupId> <artifactId>guava</artifactId> <version>${guava.version}</version> - </dependency> - <dependency> - <groupId>com.google.code.findbugs</groupId> - <artifactId>jsr305</artifactId> - <version>${jsr305.version}</version> + <exclusions> + <exclusion> + <groupId>com.google.code.findbugs</groupId> + <artifactId>jsr305</artifactId> + </exclusion> + </exclusions> </dependency> <dependency> <groupId>commons-collections</groupId> @@ -1744,7 +1766,6 @@ <dependency> <groupId>com.github.stephenc.findbugs</groupId> <artifactId>findbugs-annotations</artifactId> - <version>${findbugs-annotations}</version> <scope>compile</scope> </dependency> <dependency> @@ -2029,6 +2050,10 @@ <groupId>io.netty</groupId> <artifactId>netty</artifactId> </exclusion> + <exclusion> + <groupId>com.google.code.findbugs</groupId> + <artifactId>jsr305</artifactId> + </exclusion> </exclusions> </dependency> <dependency> @@ -2059,6 +2084,10 @@ <groupId>io.netty</groupId> <artifactId>netty</artifactId> </exclusion> + <exclusion> + <groupId>com.google.code.findbugs</groupId> + <artifactId>jsr305</artifactId> + </exclusion> </exclusions> </dependency> <dependency> @@ -2195,6 +2224,10 @@ <groupId>io.netty</groupId> <artifactId>netty</artifactId> </exclusion> + <exclusion> + <groupId>com.google.code.findbugs</groupId> + <artifactId>jsr305</artifactId> + </exclusion> </exclusions> </dependency> <dependency> @@ -2230,6 +2263,10 @@ <groupId>io.netty</groupId> <artifactId>netty</artifactId> </exclusion> + <exclusion> + <groupId>com.google.code.findbugs</groupId> + <artifactId>jsr305</artifactId> + </exclusion> </exclusions> </dependency> <dependency>
