Repository: kylin
Updated Branches:
  refs/heads/master 48f3fb195 -> febbf2623


KYLIN-2846 update unit test


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

Branch: refs/heads/master
Commit: febbf26230dec27eebe0d6568e2820ab26596b57
Parents: dbf59e5
Author: shaofengshi <[email protected]>
Authored: Sat Oct 21 23:25:41 2017 +0800
Committer: shaofengshi <[email protected]>
Committed: Mon Oct 23 17:16:27 2017 +0800

----------------------------------------------------------------------
 .../src/main/java/org/apache/kylin/cube/CubeManager.java  |  4 +++-
 .../test/java/org/apache/kylin/cube/CubeManagerTest.java  | 10 ++++++++++
 2 files changed, 13 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/febbf262/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
----------------------------------------------------------------------
diff --git a/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java 
b/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
index e24aa33..388c840 100755
--- a/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
@@ -679,7 +679,9 @@ public class CubeManager implements IRealizationProvider {
         Random ran = new Random();
         do {
             StringBuffer sb = new StringBuffer();
-            sb.append(namespace).append(":");
+            if ((namespace.equals("default") || namespace.equals("")) == 
false) {
+                sb.append(namespace).append(":");
+            }
             sb.append(namePrefix);
             for (int i = 0; i < HBASE_TABLE_LENGTH; i++) {
                 sb.append(ALPHA_NUM.charAt(ran.nextInt(ALPHA_NUM.length())));

http://git-wip-us.apache.org/repos/asf/kylin/blob/febbf262/core-cube/src/test/java/org/apache/kylin/cube/CubeManagerTest.java
----------------------------------------------------------------------
diff --git a/core-cube/src/test/java/org/apache/kylin/cube/CubeManagerTest.java 
b/core-cube/src/test/java/org/apache/kylin/cube/CubeManagerTest.java
index d8d48f7..3f68bab 100644
--- a/core-cube/src/test/java/org/apache/kylin/cube/CubeManagerTest.java
+++ b/core-cube/src/test/java/org/apache/kylin/cube/CubeManagerTest.java
@@ -314,8 +314,18 @@ public class CubeManagerTest extends 
LocalFileMetadataTestCase {
         } finally {
             System.clearProperty("kylin.storage.hbase.table-name-prefix");
         }
+
+        System.setProperty("kylin.storage.hbase.namespace", "MYSPACE");
+        try {
+            CubeManager mgr = CubeManager.getInstance(getTestConfig());
+            String tablename = mgr.generateStorageLocation();
+            assertTrue(tablename.startsWith("MYSPACE:"));
+        } finally {
+            System.clearProperty("kylin.storage.hbase.namespace");
+        }
     }
 
+
     public CubeDescManager getCubeDescManager() {
         return CubeDescManager.getInstance(getTestConfig());
     }

Reply via email to