hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From st...@apache.org
Subject hbase git commit: HBASE-13270 Setter for Result#getStats is #addResults; confusing!
Date Thu, 09 Apr 2015 17:45:08 GMT
Repository: hbase
Updated Branches:
  refs/heads/branch-1 562859f4a -> c73f1a518


HBASE-13270 Setter for Result#getStats is #addResults; confusing!

Signed-off-by: stack <stack@apache.org>


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

Branch: refs/heads/branch-1
Commit: c73f1a5183f141d7b58c5d64c439aa0f9aff0b98
Parents: 562859f
Author: Mikhail Antonov <olorinbant@gmail.com>
Authored: Thu Apr 9 01:02:15 2015 -0700
Committer: stack <stack@apache.org>
Committed: Thu Apr 9 10:44:58 2015 -0700

----------------------------------------------------------------------
 .../main/java/org/apache/hadoop/hbase/client/Result.java  | 10 ++++++++++
 1 file changed, 10 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hbase/blob/c73f1a51/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java
----------------------------------------------------------------------
diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java
index 10d5174..bfec680 100644
--- a/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java
+++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/client/Result.java
@@ -982,15 +982,25 @@ public class Result implements CellScannable, CellScanner {
   /**
    * Add load information about the region to the information about the result
    * @param loadStats statistics about the current region from which this was returned
+   * @deprecated use {@link #setStatistics(ClientProtos.RegionLoadStats)} instead
    * @throws UnsupportedOperationException if invoked on instance of EMPTY_RESULT
    * (which is supposed to be immutable).
    */
+  @Deprecated
   public void addResults(ClientProtos.RegionLoadStats loadStats) {
     checkReadonly();
     this.stats = loadStats;
   }
 
   /**
+   * Set load information about the region to the information about the result
+   * @param loadStats statistics about the current region from which this was returned
+   */
+  public void setStatistics(ClientProtos.RegionLoadStats loadStats) {
+    this.stats = loadStats;
+  }
+
+  /**
    * @return the associated statistics about the region from which this was returned. Can
be
    * <tt>null</tt> if stats are disabled.
    */


Mime
View raw message