(kylin) 24/35: KYLIN-5723 optimize KylinConfigExt

[xxyu]

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 4753eb92cd72d548aeef89f673262c90fe6c717e
Author: Mingming Ge <7mmi...@gmail.com>
AuthorDate: Fri Jun 9 14:32:35 2023 +0800

    KYLIN-5723 optimize KylinConfigExt
---
 .../src/main/java/org/apache/kylin/common/KylinConfigExt.java     | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigExt.java 
b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigExt.java
index dcfb0edc1e..1fbb20aca2 100644
--- a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigExt.java
+++ b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigExt.java
@@ -34,7 +34,7 @@ public class KylinConfigExt extends KylinConfig {
 
     private final Map<String, String> overrides;
     final KylinConfig base;
-    private final transient StrSubstitutor substitutor;
+    private final transient StrSubstitutor strSubstitutor;
 
     public static KylinConfigExt createInstance(KylinConfig kylinConfig, 
Map<String, String> overrides) {
         if (kylinConfig instanceof KylinConfigExt) {
@@ -52,7 +52,7 @@ public class KylinConfigExt extends KylinConfig {
         this.base = base;
         this.overrides = BCC.check(overrides);
         // overrides > env > properties
-        this.substitutor = new StrSubstitutor(new 
CompositeMapView(this.properties, STATIC_SYSTEM_ENV, this.overrides));
+        this.strSubstitutor = new StrSubstitutor(new 
CompositeMapView(this.properties, STATIC_SYSTEM_ENV, this.overrides));
     }
 
     private KylinConfigExt(KylinConfigExt ext, Map<String, String> overrides) {
@@ -81,14 +81,14 @@ public class KylinConfigExt extends KylinConfig {
     public Map<String, String> getReadonlyProperties() {
         HashMap<String, String> config = Maps.newHashMap();
         for (Map.Entry<String, String> entry : this.overrides.entrySet()) {
-            config.put(entry.getKey(), substitutor.replace(entry.getValue()));
+            config.put(entry.getKey(), 
strSubstitutor.replace(entry.getValue()));
         }
         return config;
     }
 
     @Override
     protected StrSubstitutor getSubstitutor() {
-        return this.substitutor;
+        return this.strSubstitutor;
     }
 
     public Map<String, String> getExtendedOverrides() {