You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2017/05/16 08:25:06 UTC
kylin git commit: minor fix UT
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 <li...@apache.org>
Authored: Tue May 16 16:24:54 2017 +0800
Committer: Li Yang <li...@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();
}