git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread aleksey
Updated Branches:
  refs/heads/cassandra-1.2.0 660016658 - f56ea8b0d


cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/cassandra-1.2.0
Commit: f56ea8b0da4ec5f26a540363b683b2a2f8221101
Parents: 6600166
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:29:26 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')



git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread aleksey
Updated Branches:
  refs/heads/cassandra-1.2 660016658 - c57fc3bd8


cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/cassandra-1.2
Commit: c57fc3bd86707a723b62492a077261b39f99c67d
Parents: 6600166
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:31:01 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/c57fc3bd/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/c57fc3bd/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')



git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread aleksey
Updated Branches:
  refs/heads/trunk 805e5cdd5 - aeef6497c


cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/trunk
Commit: aeef6497ced0745ff1f266188a862d62326f087a
Parents: 805e5cd
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:32:32 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/aeef6497/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/aeef6497/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')



[4/4] git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread dbrosius
cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/trunk
Commit: c57fc3bd86707a723b62492a077261b39f99c67d
Parents: 6600166
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:31:01 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/c57fc3bd/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/c57fc3bd/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')



[4/4] git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread slebresne
cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/cassandra-1.2
Commit: f56ea8b0da4ec5f26a540363b683b2a2f8221101
Parents: 6600166
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:29:26 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')



[5/5] git commit: cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by brandonwilliams for CASSANDRA-4913

2012-11-12 Thread slebresne
cqlsh: fix DESCRIBE command patch by Aleksey Yeschenko; reviewed by 
brandonwilliams for CASSANDRA-4913


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

Branch: refs/heads/trunk
Commit: f56ea8b0da4ec5f26a540363b683b2a2f8221101
Parents: 6600166
Author: Aleksey Yeschenko alek...@apache.org
Authored: Tue Nov 13 01:06:57 2012 +0300
Committer: Aleksey Yeschenko alek...@apache.org
Committed: Tue Nov 13 01:29:26 2012 +0300

--
 CHANGES.txt |1 +
 bin/cqlsh   |   25 +++--
 2 files changed, 12 insertions(+), 14 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/CHANGES.txt
--
diff --git a/CHANGES.txt b/CHANGES.txt
index 0ac5b66..7d4882d 100644
--- a/CHANGES.txt
+++ b/CHANGES.txt
@@ -1,4 +1,5 @@
 1.2-rc1
+ * fix cqlsh DESCRIBE command (CASSANDRA-4913)
  * save truncation position in system table (CASSANDRA-4906)
  * Move CompressionMetadata off-heap (CASSANDRA-4937)
  * allow CLI to GET cql3 columnfamily data (CASSANDRA-4924)

http://git-wip-us.apache.org/repos/asf/cassandra/blob/f56ea8b0/bin/cqlsh
--
diff --git a/bin/cqlsh b/bin/cqlsh
index 27eef7b..53eef8a 100755
--- a/bin/cqlsh
+++ b/bin/cqlsh
@@ -627,10 +627,7 @@ class Shell(cmd.Cmd):
 raise ColumnFamilyNotFound(Unconfigured column family %r % (cfname,))
 
 def get_columnfamily_names(self, ksname=None):
-if self.cqlver_atleast(3) and ksname not in SYSTEM_KEYSPACES:
-# since cql3 tables may be left out of thrift results, but
-# info on tables in system keyspaces still aren't included
-# in system.schema_*
+if self.cqlver_atleast(3):
 return self.get_columnfamily_names_cql3(ksname=ksname)
 return [c.name for c in self.get_columnfamilies(ksname)]
 
@@ -730,7 +727,7 @@ class Shell(cmd.Cmd):
 cf_q = select columnfamily from system.schema_columnfamilies
where keyspace=:ks
 self.cursor.execute(cf_q, {'ks': ksname})
-return [row[0] for row in self.cursor.fetchall()]
+return [str(row[0]) for row in self.cursor.fetchall()]
 
 def get_columnfamily_layout(self, ksname, cfname):
 if ksname is None:
@@ -1156,12 +1153,13 @@ class Shell(cmd.Cmd):
 out.write(\n  AND strategy_options:%s = %s % (opname, 
self.cql_protect_value(opval)))
 out.write(';\n')
 
-if ksdef.cf_defs:
+cfs = self.get_columnfamily_names(ksname)
+if cfs:
 out.write('\nUSE %s;\n' % ksname)
-for cf in ksdef.cf_defs:
+for cf in cfs:
 out.write('\n')
 # yes, cf might be looked up again. oh well.
-self.print_recreate_columnfamily(ksdef.name, cf.name, out)
+self.print_recreate_columnfamily(ksdef.name, cf, out)
 
 def print_recreate_columnfamily(self, ksname, cfname, out):
 
@@ -1327,20 +1325,19 @@ class Shell(cmd.Cmd):
 print
 
 def describe_columnfamilies(self, ksname):
+print
 if ksname is None:
 for k in self.get_keyspaces():
 print 'Keyspace %s' % (k.name,)
-print '-%s\n' % ('-' * len(k.name))
-cmd.Cmd.columnize(self, [c.name for c in k.cf_defs])
+print '-%s' % ('-' * len(k.name))
+cmd.Cmd.columnize(self, self.get_columnfamily_names(k.name))
 print
 else:
-names = self.get_columnfamily_names(ksname)
-print
-cmd.Cmd.columnize(self, names)
+cmd.Cmd.columnize(self, self.get_columnfamily_names(ksname))
 print
 
 def describe_cluster(self):
-print 'Cluster: %s' % self.get_cluster_name()
+print '\nCluster: %s' % self.get_cluster_name()
 p = trim_if_present(self.get_partitioner(), 
'org.apache.cassandra.dht.')
 print 'Partitioner: %s' % p
 snitch = trim_if_present(self.get_snitch(), 
'org.apache.cassandra.locator.')