You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2013/08/18 19:54:49 UTC
svn commit: r1515184 - in /hbase/branches/0.95/hbase-server/src:
main/java/org/apache/hadoop/hbase/master/HMaster.java
test/ruby/hbase/admin_test.rb
Author: apurtell
Date: Sun Aug 18 17:54:48 2013
New Revision: 1515184
URL: http://svn.apache.org/r1515184
Log:
HBASE-9251. list_namespace_tables seems to fail
Modified:
hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
hbase/branches/0.95/hbase-server/src/test/ruby/hbase/admin_test.rb
Modified: hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java?rev=1515184&r1=1515183&r2=1515184&view=diff
==============================================================================
--- hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java (original)
+++ hbase/branches/0.95/hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java Sun Aug 18 17:54:48 2013
@@ -3127,7 +3127,7 @@ MasterServices, Server {
public List<TableName> listTableNamesByNamespace(String name) throws IOException {
List<TableName> tableNames = Lists.newArrayList();
for (HTableDescriptor descriptor: tableDescriptors.getByNamespace(name).values()) {
- tableNames.add(TableName.valueOf(name, descriptor.getNameAsString()));
+ tableNames.add(descriptor.getTableName());
}
return tableNames;
}
Modified: hbase/branches/0.95/hbase-server/src/test/ruby/hbase/admin_test.rb
URL: http://svn.apache.org/viewvc/hbase/branches/0.95/hbase-server/src/test/ruby/hbase/admin_test.rb?rev=1515184&r1=1515183&r2=1515184&view=diff
==============================================================================
--- hbase/branches/0.95/hbase-server/src/test/ruby/hbase/admin_test.rb (original)
+++ hbase/branches/0.95/hbase-server/src/test/ruby/hbase/admin_test.rb Sun Aug 18 17:54:48 2013
@@ -77,6 +77,14 @@ module Hbase
assert(!admin.list.member?('hbase:meta'))
end
+ define_test "list_namespace_tables for the system namespace should return a list of tables" do
+ assert(admin.list_namespace_tables('hbase').count > 0)
+ end
+
+ define_test "list_namespace_tables for the default namespace should return a list of tables" do
+ assert(admin.list_namespace_tables('default').count > 0)
+ end
+
#-------------------------------------------------------------------------------
define_test "flush should work" do