Repository: kylin
Updated Branches:
  refs/heads/KYLIN-2624 faa27a5f3 -> f51a02026


minor fix UT


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

Branch: refs/heads/KYLIN-2624
Commit: f51a0202653ede7c70688a7174aa3ea3d1af5f4a
Parents: faa27a5
Author: Li Yang <liy...@apache.org>
Authored: Tue May 16 16:24:54 2017 +0800
Committer: Li Yang <liy...@apache.org>
Committed: Tue May 16 16:24:54 2017 +0800

----------------------------------------------------------------------
 .../test/java/org/apache/kylin/common/KylinConfigTest.java   | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/f51a0202/core-common/src/test/java/org/apache/kylin/common/KylinConfigTest.java
----------------------------------------------------------------------
diff --git 
a/core-common/src/test/java/org/apache/kylin/common/KylinConfigTest.java 
b/core-common/src/test/java/org/apache/kylin/common/KylinConfigTest.java
index 3976c6c..ea243a4 100644
--- a/core-common/src/test/java/org/apache/kylin/common/KylinConfigTest.java
+++ b/core-common/src/test/java/org/apache/kylin/common/KylinConfigTest.java
@@ -95,7 +95,7 @@ public class KylinConfigTest extends 
HotLoadKylinPropertiesTestCase {
         assertEquals("testMetaPrefix", config.getMetadataUrlPrefix());
 
         config.setMetadataUrl("/kylin/temp");
-        assertEquals(default_metadata_prefix, config.getMetadataUrlPrefix());
+        assertEquals("/kylin/temp", config.getMetadataUrlPrefix());
     }
 
     @Test
@@ -107,21 +107,21 @@ public class KylinConfigTest extends 
HotLoadKylinPropertiesTestCase {
         KylinConfig sysConfig = KylinConfig.getInstanceFromEnv();
         sysConfig.setMetadataUrl(metadata1);
 
-        assertEquals(metadata1, 
KylinConfig.getInstanceFromEnv().getMetadataUrl());
+        assertEquals(metadata1, 
KylinConfig.getInstanceFromEnv().getMetadataUrl().toString());
 
         // test thread-local override
         KylinConfig threadConfig = KylinConfig.createKylinConfig(new 
Properties());
         threadConfig.setMetadataUrl(metadata2);
         KylinConfig.setKylinConfigThreadLocal(threadConfig);
 
-        assertEquals(metadata2, 
KylinConfig.getInstanceFromEnv().getMetadataUrl());
+        assertEquals(metadata2, 
KylinConfig.getInstanceFromEnv().getMetadataUrl().toString());
 
         // other threads still use system KylinConfig
         new Thread(new Runnable() {
             @Override
             public void run() {
                 System.out.println("Started new thread.");
-                assertEquals(metadata1, 
KylinConfig.getInstanceFromEnv().getMetadataUrl());
+                assertEquals(metadata1, 
KylinConfig.getInstanceFromEnv().getMetadataUrl().toString());
             }
         }).start();
     }

Reply via email to