You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2014/10/29 21:11:52 UTC

git commit: HBASE-12355 Update maven plugins

Repository: hbase
Updated Branches:
  refs/heads/master 0fa43bd57 -> fba164a1d


HBASE-12355 Update maven plugins


Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/fba164a1
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/fba164a1
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/fba164a1

Branch: refs/heads/master
Commit: fba164a1d1f1d38827824a1903c3b7bde18f486a
Parents: 0fa43bd
Author: stack <st...@apache.org>
Authored: Wed Oct 29 13:11:44 2014 -0700
Committer: stack <st...@apache.org>
Committed: Wed Oct 29 13:11:44 2014 -0700

----------------------------------------------------------------------
 hbase-checkstyle/pom.xml                        |  3 +-
 .../org/apache/hadoop/hbase/JMXListener.java    |  2 +-
 .../ZKSplitLogManagerCoordination.java          |  4 +-
 .../org/apache/hadoop/hbase/ipc/RpcServer.java  |  2 +-
 .../hbase/master/balancer/BaseLoadBalancer.java |  2 +-
 .../apache/hadoop/hbase/quotas/QuotaCache.java  |  2 +
 .../DelimitedKeyPrefixRegionSplitPolicy.java    | 22 ++++-----
 .../KeyPrefixRegionSplitPolicy.java             | 50 ++++++++++----------
 .../hbase/regionserver/wal/HLogSplitter.java    |  4 +-
 .../hbase/security/access/AccessController.java |  4 +-
 .../hadoop/hbase/util/CompressionTest.java      |  2 +-
 .../hadoop/hbase/util/MultiHConnection.java     | 20 ++++----
 hbase-thrift/pom.xml                            | 24 ++++------
 pom.xml                                         | 29 ++++--------
 14 files changed, 72 insertions(+), 98 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-checkstyle/pom.xml
----------------------------------------------------------------------
diff --git a/hbase-checkstyle/pom.xml b/hbase-checkstyle/pom.xml
index 9387dce..6f3d71c 100644
--- a/hbase-checkstyle/pom.xml
+++ b/hbase-checkstyle/pom.xml
@@ -53,6 +53,7 @@
       </configuration>
     </plugin>
     <plugin>
+      <groupId>org.apache.maven.plugins</groupId>
       <artifactId>maven-surefire-plugin</artifactId>
       <executions>
         <execution>
@@ -69,4 +70,4 @@
     </plugin>
   </plugins>
 </build>
-</project>
\ No newline at end of file
+</project>

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/JMXListener.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/JMXListener.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/JMXListener.java
index 7a1ea11..6a115ba 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/JMXListener.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/JMXListener.java
@@ -51,7 +51,7 @@ public class JMXListener implements Coprocessor {
   public static final Log LOG = LogFactory.getLog(JMXListener.class);
   public static final String RMI_REGISTRY_PORT_CONF_KEY = ".rmi.registry.port";
   public static final String RMI_CONNECTOR_PORT_CONF_KEY = ".rmi.connector.port";
-  public static int defRMIRegistryPort = 10102;
+  public static final int defRMIRegistryPort = 10102;
 
   /**
    * workaround for HBASE-11146

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
index 0fb5c59..0f8baa3 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
@@ -98,8 +98,6 @@ public class ZKSplitLogManagerCoordination extends ZooKeeperListener implements
 
   SplitLogManagerDetails details;
 
-  private final Stoppable stopper = null;
-
   // When lastRecoveringNodeCreationTime is older than the following threshold, we'll check
   // whether to GC stale recovering znodes
   private volatile long lastRecoveringNodeCreationTime = 0;
@@ -669,7 +667,7 @@ public class ZKSplitLogManagerCoordination extends ZooKeeperListener implements
         } catch (InterruptedException e) {
           throw new InterruptedIOException();
         }
-      } while ((--retries) > 0 && (!this.stopper.isStopped()));
+      } while ((--retries) > 0);
     }
   }
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java
index e3654cb..1799d80 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/ipc/RpcServer.java
@@ -146,7 +146,7 @@ import com.google.protobuf.TextFormat;
  * and keep taking while the server is up.
  *
  * CallRunner#run executes the call.  When done, asks the included Call to put itself on new
- * queue for {@link Responder} to pull from and return result to client.
+ * queue for Responder to pull from and return result to client.
  *
  * @see RpcClient
  */

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
index a5e110b..0a2d7b0 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/balancer/BaseLoadBalancer.java
@@ -526,7 +526,7 @@ public abstract class BaseLoadBalancer implements LoadBalancer {
 
     @edu.umd.cs.findbugs.annotations.SuppressWarnings(value="NM_FIELD_NAMING_CONVENTION",
         justification="Mistake. Too disruptive to change now")
-    public static Action NullAction = new Action(Type.NULL);
+    public static final Action NullAction = new Action(Type.NULL);
 
     public void doAction(Action action) {
       switch (action.type) {

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/QuotaCache.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/QuotaCache.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/QuotaCache.java
index b67dc68..c44a737 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/QuotaCache.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/quotas/QuotaCache.java
@@ -209,6 +209,8 @@ public class QuotaCache implements Stoppable {
     }
 
     @Override
+    @edu.umd.cs.findbugs.annotations.SuppressWarnings(value="GC_UNRELATED_TYPES",
+      justification="I do not understand why the complaints, it looks good to me -- FIX")
     protected void chore() {
       // Prefetch online tables/namespaces
       for (TableName table: QuotaCache.this.rsServices.getOnlineTables()) {

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DelimitedKeyPrefixRegionSplitPolicy.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DelimitedKeyPrefixRegionSplitPolicy.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DelimitedKeyPrefixRegionSplitPolicy.java
index da3056c..36e7929 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DelimitedKeyPrefixRegionSplitPolicy.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DelimitedKeyPrefixRegionSplitPolicy.java
@@ -50,20 +50,14 @@ public class DelimitedKeyPrefixRegionSplitPolicy extends IncreasingToUpperBoundR
   @Override
   protected void configureForRegion(HRegion region) {
     super.configureForRegion(region);
-    if (region != null) {
-
-      // read the prefix length from the table descriptor
-      String delimiterString = region.getTableDesc().getValue(
-          DELIMITER_KEY);
-      if (delimiterString == null || delimiterString.length() == 0) {
-        LOG.error(DELIMITER_KEY + " not specified for table "
-            + region.getTableDesc().getTableName()
-            + ". Using default RegionSplitPolicy");
-        return;
-      }
-
-      delimiter = Bytes.toBytes(delimiterString);
+    // read the prefix length from the table descriptor
+    String delimiterString = region.getTableDesc().getValue(DELIMITER_KEY);
+    if (delimiterString == null || delimiterString.length() == 0) {
+      LOG.error(DELIMITER_KEY + " not specified for table " + region.getTableDesc().getTableName() +
+        ". Using default RegionSplitPolicy");
+      return;
     }
+    delimiter = Bytes.toBytes(delimiterString);
   }
 
   @Override
@@ -85,4 +79,4 @@ public class DelimitedKeyPrefixRegionSplitPolicy extends IncreasingToUpperBoundR
       return splitPoint;
     }
   }
-}
+}
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/KeyPrefixRegionSplitPolicy.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/KeyPrefixRegionSplitPolicy.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/KeyPrefixRegionSplitPolicy.java
index dd6afd2..b987a88 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/KeyPrefixRegionSplitPolicy.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/KeyPrefixRegionSplitPolicy.java
@@ -43,36 +43,34 @@ public class KeyPrefixRegionSplitPolicy extends IncreasingToUpperBoundRegionSpli
   @Override
   protected void configureForRegion(HRegion region) {
     super.configureForRegion(region);
-    if (region != null) {
-      prefixLength = 0;
+    prefixLength = 0;
 
-      // read the prefix length from the table descriptor
-      String prefixLengthString = region.getTableDesc().getValue(
-          PREFIX_LENGTH_KEY);
+    // read the prefix length from the table descriptor
+    String prefixLengthString = region.getTableDesc().getValue(
+        PREFIX_LENGTH_KEY);
+    if (prefixLengthString == null) {
+      //read the deprecated value
+      prefixLengthString = region.getTableDesc().getValue(PREFIX_LENGTH_KEY_DEPRECATED);
       if (prefixLengthString == null) {
-        //read the deprecated value
-        prefixLengthString = region.getTableDesc().getValue(PREFIX_LENGTH_KEY_DEPRECATED);
-        if (prefixLengthString == null) {
-          LOG.error(PREFIX_LENGTH_KEY + " not specified for table "
-              + region.getTableDesc().getTableName()
-              + ". Using default RegionSplitPolicy");
-          return;
-        }
-      }
-      try {
-        prefixLength = Integer.parseInt(prefixLengthString);
-      } catch (NumberFormatException nfe) {
-        /* Differentiate NumberFormatException from an invalid value range reported below. */
-        LOG.error("Number format exception when parsing " + PREFIX_LENGTH_KEY + " for table "
-            + region.getTableDesc().getTableName() + ":"
-            + prefixLengthString + ". " + nfe);
+        LOG.error(PREFIX_LENGTH_KEY + " not specified for table "
+            + region.getTableDesc().getTableName()
+            + ". Using default RegionSplitPolicy");
         return;
       }
-      if (prefixLength <= 0) {
-        LOG.error("Invalid value for " + PREFIX_LENGTH_KEY + " for table "
-            + region.getTableDesc().getTableName() + ":"
-            + prefixLengthString + ". Using default RegionSplitPolicy");
-      }
+    }
+    try {
+      prefixLength = Integer.parseInt(prefixLengthString);
+    } catch (NumberFormatException nfe) {
+      /* Differentiate NumberFormatException from an invalid value range reported below. */
+      LOG.error("Number format exception when parsing " + PREFIX_LENGTH_KEY + " for table "
+          + region.getTableDesc().getTableName() + ":"
+          + prefixLengthString + ". " + nfe);
+      return;
+    }
+    if (prefixLength <= 0) {
+      LOG.error("Invalid value for " + PREFIX_LENGTH_KEY + " for table "
+          + region.getTableDesc().getTableName() + ":"
+          + prefixLengthString + ". Using default RegionSplitPolicy");
     }
   }
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
index dc46ff1..1f81874 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/HLogSplitter.java
@@ -2048,9 +2048,7 @@ public class HLogSplitter {
         }
         ((Put) m).add(tmpNewCell);
       }
-      if (m != null) {
-        m.setDurability(durability);
-      }
+      m.setDurability(durability);
       previousCell = cell;
     }
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
index dd13e0c..ab1b4a8 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
@@ -1296,7 +1296,7 @@ public class AccessController extends BaseMasterAndRegionObserver
             ourFilter = new FilterList(FilterList.Operator.MUST_PASS_ALL,
               Lists.newArrayList(ourFilter, filter));
           }
-          authResult.setAllowed(true);;
+          authResult.setAllowed(true);
           authResult.setReason("Access allowed with filter");
           switch (opType) {
           case GET:
@@ -1322,7 +1322,7 @@ public class AccessController extends BaseMasterAndRegionObserver
           ourFilter = new FilterList(FilterList.Operator.MUST_PASS_ALL,
             Lists.newArrayList(ourFilter, filter));
         }
-        authResult.setAllowed(true);;
+        authResult.setAllowed(true);
         authResult.setReason("Access allowed with filter");
         switch (opType) {
         case GET:

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
index 850dc02..abf3a0d 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
@@ -127,7 +127,7 @@ public class CompressionTest {
     // Write any-old Cell...
     final byte [] rowKey = Bytes.toBytes("compressiontestkey");
     Cell c = CellUtil.createCell(rowKey, Bytes.toBytes("compressiontestval"));
-    writer.append(c);;
+    writer.append(c);
     writer.appendFileInfo(Bytes.toBytes("compressioninfokey"), Bytes.toBytes("compressioninfoval"));
     writer.close();
     Cell cc = null;

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-server/src/main/java/org/apache/hadoop/hbase/util/MultiHConnection.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/MultiHConnection.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/MultiHConnection.java
index 1dc9c31..81678aa 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/MultiHConnection.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/MultiHConnection.java
@@ -77,20 +77,18 @@ public class MultiHConnection {
   public void close() {
     synchronized (hConnectionsLock) {
       if (hConnections != null) {
-        if (hConnections != null) {
-          for (Connection conn : hConnections) {
-            if (conn != null) {
-              try {
-                conn.close();
-              } catch (IOException e) {
-                LOG.info("Got exception in closing connection", e);
-              } finally {
-                conn = null;
-              }
+        for (Connection conn : hConnections) {
+          if (conn != null) {
+            try {
+              conn.close();
+            } catch (IOException e) {
+              LOG.info("Got exception in closing connection", e);
+            } finally {
+              conn = null;
             }
           }
-          hConnections = null;
         }
+        hConnections = null;
       }
     }
     if (this.batchPool != null && !this.batchPool.isShutdown()) {

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/hbase-thrift/pom.xml
----------------------------------------------------------------------
diff --git a/hbase-thrift/pom.xml b/hbase-thrift/pom.xml
index 0f64d00..758091a 100644
--- a/hbase-thrift/pom.xml
+++ b/hbase-thrift/pom.xml
@@ -93,6 +93,15 @@
             </goals>
             <configuration>
               <skip>true</skip>
+             <properties>
+               <property>
+                 <name>listener</name>
+                 <value>org.apache.hadoop.hbase.ServerResourceCheckerJUnitListener</value>
+               </property>
+             </properties>
+             <systemPropertyVariables>
+               <test.build.webapps>target/test-classes/webapps</test.build.webapps>
+             </systemPropertyVariables>
             </configuration>
           </execution>
         </executions>
@@ -159,21 +168,6 @@
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
       </plugin>
-      <!-- Testing plugins -->
-      <plugin>
-        <artifactId>maven-surefire-plugin</artifactId>
-        <configuration>
-          <properties>
-            <property>
-              <name>listener</name>
-              <value>org.apache.hadoop.hbase.ServerResourceCheckerJUnitListener</value>
-            </property>
-          </properties>
-          <systemPropertyVariables>
-            <test.build.webapps>target/test-classes/webapps</test.build.webapps>
-          </systemPropertyVariables>
-        </configuration>
-      </plugin>
     </plugins>
   </build>
 

http://git-wip-us.apache.org/repos/asf/hbase/blob/fba164a1/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 076a446..c0ced03 100644
--- a/pom.xml
+++ b/pom.xml
@@ -416,16 +416,6 @@
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-release-plugin</artifactId>
-          <version>2.4.1</version>
-          <!--Making a release I've been using mvn 3.0 and specifying the apache-release
-              profile on the command line as follows:
-
-              $ ~/bin/mvn/bin/mvn  -Papache-release release:perform
-
-              I've also been kiling the release:prepare step mid-way to check the
-              release.properties it generates at the top-level.  Sometimes it refers
-              to HEAD rather than to the svn branch.
-           -->
           <configuration>
             <!--You need this profile. It'll sign your artifacts.
                 I'm not sure if this config. actually works though.
@@ -440,6 +430,7 @@
           </configuration>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-compiler-plugin</artifactId>
           <version>3.2</version>
           <configuration>
@@ -469,6 +460,7 @@
         </plugin>
         <!-- Test oriented plugins -->
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-plugin</artifactId>
           <version>${surefire.version}</version>
           <dependencies>
@@ -514,10 +506,12 @@
           </executions>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-surefire-report-plugin</artifactId>
           <version>${surefire.version}</version>
         </plugin>
         <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-clean-plugin</artifactId>
           <configuration>
             <filesets>
@@ -532,7 +526,8 @@
         <plugin>
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>findbugs-maven-plugin</artifactId>
-          <version>${findbugs-maven-plugin.version}</version>
+          <version>3.0.0</version>
+          <!--NOTE: Findbugs 3.0.0 requires jdk7-->
           <configuration>
             <excludeFilterFile>${project.basedir}/../dev-support/findbugs-exclude.xml</excludeFilterFile>
             <findbugsXmlOutput>true</findbugsXmlOutput>
@@ -543,7 +538,7 @@
         <plugin>
           <groupId>org.codehaus.mojo</groupId>
           <artifactId>build-helper-maven-plugin</artifactId>
-          <version>1.5</version>
+          <version>1.9.1</version>
         </plugin>
         <plugin>
           <artifactId>maven-antrun-plugin</artifactId>
@@ -599,7 +594,7 @@
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-eclipse-plugin</artifactId>
-          <version>2.8</version>
+          <version>2.9</version>
         </plugin>
         <!--This plugin's configuration is used to store Eclipse m2e settings only. It
           has no influence on the Maven build itself. m2e does not provide any safeguards against
@@ -616,7 +611,7 @@
           <!-- excludes are inherited -->
           <groupId>org.apache.rat</groupId>
           <artifactId>apache-rat-plugin</artifactId>
-          <version>0.8</version>
+          <version>0.11</version>
           <configuration>
             <excludes>
               <exclude>**/*.versionsBackup</exclude>
@@ -843,7 +838,6 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-resources-plugin</artifactId>
-        <version>${maven.resources.plugin.version}</version>
         <!--$NO-MVN-MAN-VER$ -->
         <inherited>false</inherited>
         <executions>
@@ -891,7 +885,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-site-plugin</artifactId>
-        <version>${maven.site.version}</version>
+        <version>3.4</version>
         <inherited>false</inherited>
         <dependencies>
           <dependency>
@@ -983,11 +977,8 @@
     <maven.assembly.version>2.4</maven.assembly.version>
     <maven.antrun.version>1.6</maven.antrun.version>
     <jamon.plugin.version>2.3.4</jamon.plugin.version>
-    <findbugs-maven-plugin.version>2.5.2</findbugs-maven-plugin.version>
     <findbugs-annotations>1.3.9-1</findbugs-annotations>
-    <maven.site.version>3.3</maven.site.version>
     <javadoc.version>2.9</javadoc.version>
-    <maven.resources.plugin.version>2.6</maven.resources.plugin.version>
     <!-- General Packaging -->
     <package.prefix>/usr</package.prefix>
     <package.conf.dir>/etc/hbase</package.conf.dir>