From 1deadb60d1a684cbc1a45cfeb6eee176ad93958f Mon Sep 17 00:00:00 2001 From: Mikhail Antonov Date: Thu, 9 Apr 2015 01:02:15 -0700 Subject: [PATCH] HBASE-13270 Setter for Result#getStats is #addResults; confusing! Signed-off-by: stack --- .../java/org/apache/hadoop/hbase/client/Result.java | 10 ++++++++++ 1 file changed, 10 insertions(+) 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 b6600a94636..5a9aff3eda2 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 @@ -907,14 +907,24 @@ 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 * null if stats are disabled.