rename getRangeKeySample to sampleKeyRange
patch by Jan Prach; reviewed by jbellis for CASSANDRA-4452


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

Branch: refs/heads/cassandra-1.1
Commit: 06bdd3ea8e6ec8ebf47b7bd813041550f99fa48b
Parents: 6f384c5
Author: Jonathan Ellis <jbel...@apache.org>
Authored: Wed Jul 25 13:12:32 2012 -0500
Committer: Jonathan Ellis <jbel...@apache.org>
Committed: Wed Jul 25 13:12:32 2012 -0500

----------------------------------------------------------------------
 CHANGES.txt                                        |    2 ++
 .../apache/cassandra/service/StorageService.java   |    2 +-
 .../cassandra/service/StorageServiceMBean.java     |    2 +-
 src/java/org/apache/cassandra/tools/NodeCmd.java   |    2 +-
 src/java/org/apache/cassandra/tools/NodeProbe.java |    4 ++--
 5 files changed, 7 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cassandra/blob/06bdd3ea/CHANGES.txt
----------------------------------------------------------------------
diff --git a/CHANGES.txt b/CHANGES.txt
index 638574c..c160d69 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,6 @@
 1.1.3
+ * (JMX) rename getRangeKeySample to sampleKeyRange to avoid returning
+   multi-MB results as an attribute (CASSANDRA-4452)
  * flush based on data size, not throughput; overwritten columns no 
    longer artificially inflate liveRatio (CASSANDRA-4399)
  * update default commitlog segment size to 32MB and total commitlog

http://git-wip-us.apache.org/repos/asf/cassandra/blob/06bdd3ea/src/java/org/apache/cassandra/service/StorageService.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/service/StorageService.java 
b/src/java/org/apache/cassandra/service/StorageService.java
index 28a3551..bfc8c81 100644
--- a/src/java/org/apache/cassandra/service/StorageService.java
+++ b/src/java/org/apache/cassandra/service/StorageService.java
@@ -3080,7 +3080,7 @@ public class StorageService implements 
IEndpointStateChangeSubscriber, StorageSe
     /**
      * #{@inheritDoc}
      */
-    public List<String> getRangeKeySample()
+    public List<String> sampleKeyRange() // do not rename to getter - see 
CASSANDRA-4452 for details
     {
         List<DecoratedKey> keys = 
keySamples(ColumnFamilyStore.allUserDefined(), getLocalPrimaryRange());
 

http://git-wip-us.apache.org/repos/asf/cassandra/blob/06bdd3ea/src/java/org/apache/cassandra/service/StorageServiceMBean.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/service/StorageServiceMBean.java 
b/src/java/org/apache/cassandra/service/StorageServiceMBean.java
index 72d03d1..0872e2b 100644
--- a/src/java/org/apache/cassandra/service/StorageServiceMBean.java
+++ b/src/java/org/apache/cassandra/service/StorageServiceMBean.java
@@ -406,7 +406,7 @@ public interface StorageServiceMBean
      *
      * @return set of Tokens as Strings
      */
-    public List<String> getRangeKeySample();
+    public List<String> sampleKeyRange();
 
     /**
      * rebuild the specified indexes

http://git-wip-us.apache.org/repos/asf/cassandra/blob/06bdd3ea/src/java/org/apache/cassandra/tools/NodeCmd.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/tools/NodeCmd.java 
b/src/java/org/apache/cassandra/tools/NodeCmd.java
index a8d3f55..b73e96a 100644
--- a/src/java/org/apache/cassandra/tools/NodeCmd.java
+++ b/src/java/org/apache/cassandra/tools/NodeCmd.java
@@ -922,7 +922,7 @@ public class NodeCmd
     private void printRangeKeySample(PrintStream outs)
     {
         outs.println("RangeKeySample: ");
-        List<String> tokenStrings = this.probe.getRangeKeySample();
+        List<String> tokenStrings = this.probe.sampleKeyRange();
         for (String tokenString : tokenStrings)
         {
             outs.println("\t" + tokenString);

http://git-wip-us.apache.org/repos/asf/cassandra/blob/06bdd3ea/src/java/org/apache/cassandra/tools/NodeProbe.java
----------------------------------------------------------------------
diff --git a/src/java/org/apache/cassandra/tools/NodeProbe.java 
b/src/java/org/apache/cassandra/tools/NodeProbe.java
index d1a615d..5c04eff 100644
--- a/src/java/org/apache/cassandra/tools/NodeProbe.java
+++ b/src/java/org/apache/cassandra/tools/NodeProbe.java
@@ -690,9 +690,9 @@ public class NodeProbe
         ssProxy.rebuild(sourceDc);
     }
 
-    public List<String> getRangeKeySample()
+    public List<String> sampleKeyRange()
     {
-        return ssProxy.getRangeKeySample();
+        return ssProxy.sampleKeyRange();
     }
 
     public void resetLocalSchema() throws IOException

Reply via email to