You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2022/12/13 10:25:23 UTC

[kylin] 16/25: KYLIN-5274 Improve performance of getSubstitutor

This is an automated email from the ASF dual-hosted git repository.

xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit a74ac29de0599ca55dcd955da0cba03070087e2e
Author: Junqing Cai <ca...@163.com>
AuthorDate: Thu Oct 13 18:27:10 2022 +0800

    KYLIN-5274 Improve performance of getSubstitutor
---
 .../test/java/org/apache/kylin/common/KylinConfigBaseTest.java    | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/core-common/src/test/java/org/apache/kylin/common/KylinConfigBaseTest.java b/src/core-common/src/test/java/org/apache/kylin/common/KylinConfigBaseTest.java
index 70c90517ab..16f653d46b 100644
--- a/src/core-common/src/test/java/org/apache/kylin/common/KylinConfigBaseTest.java
+++ b/src/core-common/src/test/java/org/apache/kylin/common/KylinConfigBaseTest.java
@@ -1372,6 +1372,14 @@ class KylinConfigBaseTest {
         config.setProperty("kylin.server.leader-race.heart-beat-timeout-rate", "1");
         Assertions.assertEquals(1.0, config.getEpochRenewTimeoutRate());
     }
+
+    @Test
+    void testGetSubstitutor() {
+        KylinConfig config = KylinConfig.getInstanceFromEnv();
+        val sub1 = config.getSubstitutor();
+        val sub2 = config.getSubstitutor();
+        Assertions.assertSame(sub1, sub2);
+    }
 }
 
 class EnvironmentUpdateUtils {