git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Repository: cassandra
Updated Branches:
  refs/heads/cassandra-1.2 d0b41d009 - 8d1acd93f


Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/cassandra-1.2
Commit: 8d1acd93fe86030bdb84bba5251f7999ba70e1b7
Parents: d0b41d0
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:33:39 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:33:39 2014 -0400

--
 CHANGES.txt| 1 +
 src/java/org/apache/cassandra/tools/NodeCmd.java   | 5 +
 src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml | 2 +-
 3 files changed, 3 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index b4498fa..8cfffad 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@
  * Shutdown batchlog executor in SS#drain() (CASSANDRA-7025)
  * Fix batchlog to account for CF truncation records (CASSANDRA-6999)
  * Fix CQLSH parsing of functions and BLOB literals (CASSANDRA-7018)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 
 
 1.2.16

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/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 6c02877..4901806 100644
--- a/src/java/org/apache/cassandra/tools/NodeCmd.java
+++ b/src/java/org/apache/cassandra/tools/NodeCmd.java
@@ -1370,11 +1370,8 @@ public class NodeCmd
 break;
 
 case REBUILD_INDEX:
-if (arguments.length  2) { badUse(rebuild_index requires 
ks and cf args); }
-if (arguments.length = 3)
+if (arguments.length = 2) { badUse(rebuild_index 
requires ks, cf and idx args); }
 probe.rebuildIndex(arguments[0], arguments[1], 
arguments[2].split(,));
-else
-probe.rebuildIndex(arguments[0], arguments[1]);
 
 break;
 

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
--
diff --git a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml 
b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
index 8fe0519..ea2f0d7 100644
--- a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
+++ b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
@@ -181,7 +181,7 @@ commands:
   Load newly placed SSTables to the system without restart.
   - name: rebuild_index keyspace cf-name idx1,idx1
 help: |
-  a full rebuilds of native secondry index for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
+  a full rebuild of native secondary indexes for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
   - name: setcachecapacity key-cache-capacity row-cache-capacity
 help: |
   Set global key and row cache capacities (in MB units).



[1/2] git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Repository: cassandra
Updated Branches:
  refs/heads/cassandra-2.0 1104653a4 - bc6f4d003


Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/cassandra-2.0
Commit: 8d1acd93fe86030bdb84bba5251f7999ba70e1b7
Parents: d0b41d0
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:33:39 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:33:39 2014 -0400

--
 CHANGES.txt| 1 +
 src/java/org/apache/cassandra/tools/NodeCmd.java   | 5 +
 src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml | 2 +-
 3 files changed, 3 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index b4498fa..8cfffad 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@
  * Shutdown batchlog executor in SS#drain() (CASSANDRA-7025)
  * Fix batchlog to account for CF truncation records (CASSANDRA-6999)
  * Fix CQLSH parsing of functions and BLOB literals (CASSANDRA-7018)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 
 
 1.2.16

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/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 6c02877..4901806 100644
--- a/src/java/org/apache/cassandra/tools/NodeCmd.java
+++ b/src/java/org/apache/cassandra/tools/NodeCmd.java
@@ -1370,11 +1370,8 @@ public class NodeCmd
 break;
 
 case REBUILD_INDEX:
-if (arguments.length  2) { badUse(rebuild_index requires 
ks and cf args); }
-if (arguments.length = 3)
+if (arguments.length = 2) { badUse(rebuild_index 
requires ks, cf and idx args); }
 probe.rebuildIndex(arguments[0], arguments[1], 
arguments[2].split(,));
-else
-probe.rebuildIndex(arguments[0], arguments[1]);
 
 break;
 

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
--
diff --git a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml 
b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
index 8fe0519..ea2f0d7 100644
--- a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
+++ b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
@@ -181,7 +181,7 @@ commands:
   Load newly placed SSTables to the system without restart.
   - name: rebuild_index keyspace cf-name idx1,idx1
 help: |
-  a full rebuilds of native secondry index for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
+  a full rebuild of native secondary indexes for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
   - name: setcachecapacity key-cache-capacity row-cache-capacity
 help: |
   Set global key and row cache capacities (in MB units).



git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Repository: cassandra
Updated Branches:
  refs/heads/cassandra-2.1 2b89cf6a8 - 743d921ae


Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/cassandra-2.1
Commit: 743d921ae8591545ff01022db9a41a84ef1edcd1
Parents: 2b89cf6
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:46:31 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:46:31 2014 -0400

--
 CHANGES.txt   |  1 +
 src/java/org/apache/cassandra/tools/NodeTool.java | 15 ---
 2 files changed, 5 insertions(+), 11 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/743d921a/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 5d15eac..d94f13b 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -46,6 +46,7 @@
  * Fix AE when closing SSTable without releasing reference (CASSANDRA-7000)
  * Clean up IndexInfo on keyspace/table drops (CASSANDRA-6924)
  * Only snapshot relative SSTables when sequential repair (CASSANDRA-7024)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 Merged from 2.0:
  * Put nodes in hibernate when join_ring is false (CASSANDRA-6961)
  * Allow compaction of system tables during startup (CASSANDRA-6913)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/743d921a/src/java/org/apache/cassandra/tools/NodeTool.java
--
diff --git a/src/java/org/apache/cassandra/tools/NodeTool.java 
b/src/java/org/apache/cassandra/tools/NodeTool.java
index edcfec3..2c8d0c2 100644
--- a/src/java/org/apache/cassandra/tools/NodeTool.java
+++ b/src/java/org/apache/cassandra/tools/NodeTool.java
@@ -2130,24 +2130,17 @@ public class NodeTool
 }
 }
 
-@Command(name = rebuild_index, description = A full rebuilds of native 
secondry index for a given column family)
+@Command(name = rebuild_index, description = A full rebuild of native 
secondary indexes for a given column family)
 public static class RebuildIndex extends NodeToolCmd
 {
-@Arguments(usage = keyspace cfname [indexName...], description 
= The keyspace and column family name followed by an optional list of index 
names (IndexNameExample: Standard3.IdxName Standard3.IdxName1))
+@Arguments(usage = keyspace cfname indexName..., description = 
The keyspace and column family name followed by a list of index names 
(IndexNameExample: Standard3.IdxName Standard3.IdxName1))
 ListString args = new ArrayList();
 
 @Override
 public void execute(NodeProbe probe)
 {
-checkArgument(args.size() = 2, rebuild_index requires ks and cf 
args);
-
-ListString indexNames = new ArrayList();
-if (args.size()  2)
-{
-indexNames.addAll(args.subList(2, args.size()));
-}
-
-probe.rebuildIndex(args.get(0), args.get(1), toArray(indexNames, 
String.class));
+checkArgument(args.size() = 3, rebuild_index requires ks, cf and 
idx args);
+probe.rebuildIndex(args.get(0), args.get(1), 
toArray(args.subList(2, args.size()), String.class));
 }
 }
 



[1/3] git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Repository: cassandra
Updated Branches:
  refs/heads/cassandra-2.1 743d921ae - 30e2bff69


Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/cassandra-2.1
Commit: 8d1acd93fe86030bdb84bba5251f7999ba70e1b7
Parents: d0b41d0
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:33:39 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:33:39 2014 -0400

--
 CHANGES.txt| 1 +
 src/java/org/apache/cassandra/tools/NodeCmd.java   | 5 +
 src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml | 2 +-
 3 files changed, 3 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index b4498fa..8cfffad 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@
  * Shutdown batchlog executor in SS#drain() (CASSANDRA-7025)
  * Fix batchlog to account for CF truncation records (CASSANDRA-6999)
  * Fix CQLSH parsing of functions and BLOB literals (CASSANDRA-7018)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 
 
 1.2.16

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/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 6c02877..4901806 100644
--- a/src/java/org/apache/cassandra/tools/NodeCmd.java
+++ b/src/java/org/apache/cassandra/tools/NodeCmd.java
@@ -1370,11 +1370,8 @@ public class NodeCmd
 break;
 
 case REBUILD_INDEX:
-if (arguments.length  2) { badUse(rebuild_index requires 
ks and cf args); }
-if (arguments.length = 3)
+if (arguments.length = 2) { badUse(rebuild_index 
requires ks, cf and idx args); }
 probe.rebuildIndex(arguments[0], arguments[1], 
arguments[2].split(,));
-else
-probe.rebuildIndex(arguments[0], arguments[1]);
 
 break;
 

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
--
diff --git a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml 
b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
index 8fe0519..ea2f0d7 100644
--- a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
+++ b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
@@ -181,7 +181,7 @@ commands:
   Load newly placed SSTables to the system without restart.
   - name: rebuild_index keyspace cf-name idx1,idx1
 help: |
-  a full rebuilds of native secondry index for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
+  a full rebuild of native secondary indexes for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
   - name: setcachecapacity key-cache-capacity row-cache-capacity
 help: |
   Set global key and row cache capacities (in MB units).



[1/5] git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Repository: cassandra
Updated Branches:
  refs/heads/trunk 462c2dcc0 - 33fa4f648


Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/trunk
Commit: 8d1acd93fe86030bdb84bba5251f7999ba70e1b7
Parents: d0b41d0
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:33:39 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:33:39 2014 -0400

--
 CHANGES.txt| 1 +
 src/java/org/apache/cassandra/tools/NodeCmd.java   | 5 +
 src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml | 2 +-
 3 files changed, 3 insertions(+), 5 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index b4498fa..8cfffad 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -7,6 +7,7 @@
  * Shutdown batchlog executor in SS#drain() (CASSANDRA-7025)
  * Fix batchlog to account for CF truncation records (CASSANDRA-6999)
  * Fix CQLSH parsing of functions and BLOB literals (CASSANDRA-7018)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 
 
 1.2.16

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/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 6c02877..4901806 100644
--- a/src/java/org/apache/cassandra/tools/NodeCmd.java
+++ b/src/java/org/apache/cassandra/tools/NodeCmd.java
@@ -1370,11 +1370,8 @@ public class NodeCmd
 break;
 
 case REBUILD_INDEX:
-if (arguments.length  2) { badUse(rebuild_index requires 
ks and cf args); }
-if (arguments.length = 3)
+if (arguments.length = 2) { badUse(rebuild_index 
requires ks, cf and idx args); }
 probe.rebuildIndex(arguments[0], arguments[1], 
arguments[2].split(,));
-else
-probe.rebuildIndex(arguments[0], arguments[1]);
 
 break;
 

http://git-wip-us.apache.org/repos/asf/cassandra/blob/8d1acd93/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
--
diff --git a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml 
b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
index 8fe0519..ea2f0d7 100644
--- a/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
+++ b/src/resources/org/apache/cassandra/tools/NodeToolHelp.yaml
@@ -181,7 +181,7 @@ commands:
   Load newly placed SSTables to the system without restart.
   - name: rebuild_index keyspace cf-name idx1,idx1
 help: |
-  a full rebuilds of native secondry index for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
+  a full rebuild of native secondary indexes for a given column family. 
IndexNameExample: Standard3.IdxName,Standard3.IdxName1
   - name: setcachecapacity key-cache-capacity row-cache-capacity
 help: |
   Set global key and row cache capacities (in MB units).



[3/5] git commit: Nodetool rebuild_index requires named indexes argument

2014-04-21 Thread dbrosius
Nodetool rebuild_index requires named indexes argument

patch by Stunnicliffe reviewed by dbrosius for cassandra-7038


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

Branch: refs/heads/trunk
Commit: 743d921ae8591545ff01022db9a41a84ef1edcd1
Parents: 2b89cf6
Author: Sam Tunnicliffe s...@beobal.com
Authored: Tue Apr 22 01:46:31 2014 -0400
Committer: Dave Brosius dbros...@mebigfatguy.com
Committed: Tue Apr 22 01:46:31 2014 -0400

--
 CHANGES.txt   |  1 +
 src/java/org/apache/cassandra/tools/NodeTool.java | 15 ---
 2 files changed, 5 insertions(+), 11 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/743d921a/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 5d15eac..d94f13b 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -46,6 +46,7 @@
  * Fix AE when closing SSTable without releasing reference (CASSANDRA-7000)
  * Clean up IndexInfo on keyspace/table drops (CASSANDRA-6924)
  * Only snapshot relative SSTables when sequential repair (CASSANDRA-7024)
+ * Require nodetool rebuild_index to specify index names (CASSANDRA-7038)
 Merged from 2.0:
  * Put nodes in hibernate when join_ring is false (CASSANDRA-6961)
  * Allow compaction of system tables during startup (CASSANDRA-6913)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/743d921a/src/java/org/apache/cassandra/tools/NodeTool.java
--
diff --git a/src/java/org/apache/cassandra/tools/NodeTool.java 
b/src/java/org/apache/cassandra/tools/NodeTool.java
index edcfec3..2c8d0c2 100644
--- a/src/java/org/apache/cassandra/tools/NodeTool.java
+++ b/src/java/org/apache/cassandra/tools/NodeTool.java
@@ -2130,24 +2130,17 @@ public class NodeTool
 }
 }
 
-@Command(name = rebuild_index, description = A full rebuilds of native 
secondry index for a given column family)
+@Command(name = rebuild_index, description = A full rebuild of native 
secondary indexes for a given column family)
 public static class RebuildIndex extends NodeToolCmd
 {
-@Arguments(usage = keyspace cfname [indexName...], description 
= The keyspace and column family name followed by an optional list of index 
names (IndexNameExample: Standard3.IdxName Standard3.IdxName1))
+@Arguments(usage = keyspace cfname indexName..., description = 
The keyspace and column family name followed by a list of index names 
(IndexNameExample: Standard3.IdxName Standard3.IdxName1))
 ListString args = new ArrayList();
 
 @Override
 public void execute(NodeProbe probe)
 {
-checkArgument(args.size() = 2, rebuild_index requires ks and cf 
args);
-
-ListString indexNames = new ArrayList();
-if (args.size()  2)
-{
-indexNames.addAll(args.subList(2, args.size()));
-}
-
-probe.rebuildIndex(args.get(0), args.get(1), toArray(indexNames, 
String.class));
+checkArgument(args.size() = 3, rebuild_index requires ks, cf and 
idx args);
+probe.rebuildIndex(args.get(0), args.get(1), 
toArray(args.subList(2, args.size()), String.class));
 }
 }