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 2016/12/01 06:03:42 UTC

hbase git commit: HBASE-17216 Fields that can safetly be made static

Repository: hbase
Updated Branches:
  refs/heads/master 1f3c6f4c0 -> 37d7b5712


HBASE-17216 Fields that can safetly be made static


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

Branch: refs/heads/master
Commit: 37d7b571287ea185b2f6d9a32e40b0fa10afcee2
Parents: 1f3c6f4
Author: John Leach <jl...@splicemachine.com>
Authored: Wed Nov 30 13:15:22 2016 -0600
Committer: Michael Stack <st...@apache.org>
Committed: Wed Nov 30 22:03:35 2016 -0800

----------------------------------------------------------------------
 .../org/apache/hadoop/hbase/client/AsyncProcess.java    |  2 +-
 .../hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java  | 12 ++++++------
 .../hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java  |  2 +-
 .../apache/hadoop/hbase/regionserver/wal/WALEdit.java   |  2 +-
 .../hbase/replication/regionserver/HFileReplicator.java |  2 +-
 5 files changed, 10 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/37d7b571/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
index 868295d..debb602 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/AsyncProcess.java
@@ -118,7 +118,7 @@ class AsyncProcess {
   private static final String THRESHOLD_TO_LOG_UNDONE_TASK_DETAILS =
       "hbase.client.threshold.log.details";
   private static final int DEFAULT_THRESHOLD_TO_LOG_UNDONE_TASK_DETAILS = 10;
-  private final int THRESHOLD_TO_LOG_REGION_DETAILS = 2;
+  private static final int THRESHOLD_TO_LOG_REGION_DETAILS = 2;
 
   /**
    * The maximum size of single RegionServer.

http://git-wip-us.apache.org/repos/asf/hbase/blob/37d7b571/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java
----------------------------------------------------------------------
diff --git a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java
index 85b33f8..5d7a379 100644
--- a/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java
+++ b/hbase-common/src/main/java/org/apache/hadoop/hbase/io/encoding/FastDiffDeltaEncoder.java
@@ -59,12 +59,12 @@ import org.apache.hadoop.hbase.util.ObjectIntPair;
  */
 @InterfaceAudience.Private
 public class FastDiffDeltaEncoder extends BufferedDataBlockEncoder {
-  final int MASK_TIMESTAMP_LENGTH = (1 << 0) | (1 << 1) | (1 << 2);
-  final int SHIFT_TIMESTAMP_LENGTH = 0;
-  final int FLAG_SAME_KEY_LENGTH = 1 << 3;
-  final int FLAG_SAME_VALUE_LENGTH = 1 << 4;
-  final int FLAG_SAME_TYPE = 1 << 5;
-  final int FLAG_SAME_VALUE = 1 << 6;
+  static final int MASK_TIMESTAMP_LENGTH = (1 << 0) | (1 << 1) | (1 << 2);
+  static final int SHIFT_TIMESTAMP_LENGTH = 0;
+  static final int FLAG_SAME_KEY_LENGTH = 1 << 3;
+  static final int FLAG_SAME_VALUE_LENGTH = 1 << 4;
+  static final int FLAG_SAME_TYPE = 1 << 5;
+  static final int FLAG_SAME_VALUE = 1 << 6;
 
   private static class FastDiffCompressionState extends CompressionState {
     byte[] timestamp = new byte[KeyValue.TIMESTAMP_SIZE];

http://git-wip-us.apache.org/repos/asf/hbase/blob/37d7b571/hbase-server/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java
index 1ac1920..a6ee6da 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/io/asyncfs/FanOutOneBlockAsyncDFSOutput.java
@@ -170,7 +170,7 @@ public class FanOutOneBlockAsyncDFSOutput implements AsyncFSOutput {
   private int capacity = 4 * 1024;
 
   // LIMIT is 128MB
-  private final int LIMIT = 128 * 1024 * 1024;
+  private static final int LIMIT = 128 * 1024 * 1024;
 
   private enum State {
     STREAMING, CLOSING, BROKEN, CLOSED

http://git-wip-us.apache.org/repos/asf/hbase/blob/37d7b571/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java
index 75c1c3e..f79fa01 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/wal/WALEdit.java
@@ -100,7 +100,7 @@ public class WALEdit implements Writable, HeapSize {
   @VisibleForTesting
   public static final byte [] BULK_LOAD = Bytes.toBytes("HBASE::BULK_LOAD");
 
-  private final int VERSION_2 = -1;
+  private static final int VERSION_2 = -1;
   private final boolean isReplay;
 
   private ArrayList<Cell> cells = null;

http://git-wip-us.apache.org/repos/asf/hbase/blob/37d7b571/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java
----------------------------------------------------------------------
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java
index cc02fb6..9eda7e4 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/HFileReplicator.java
@@ -71,7 +71,7 @@ public class HFileReplicator {
   public static final int REPLICATION_BULKLOAD_COPY_HFILES_PERTHREAD_DEFAULT = 10;
 
   private static final Log LOG = LogFactory.getLog(HFileReplicator.class);
-  private final String UNDERSCORE = "_";
+  private static final String UNDERSCORE = "_";
   private final static FsPermission PERM_ALL_ACCESS = FsPermission.valueOf("-rwxrwxrwx");
 
   private Configuration sourceClusterConf;