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 ce432fa60cdacaffaaaa8fb23b6952e4051c9cab Author: Junqing Cai <caicai121...@163.com> AuthorDate: Mon Oct 17 10:11:33 2022 +0800 KYLIN-5274 Fix sonar --- .../src/main/java/org/apache/kylin/common/KylinConfigBase.java | 4 ++-- .../main/java/org/apache/kylin/common/KylinExternalConfigLoader.java | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java index 33f7f0b5f1..e64192aa3f 100644 --- a/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java +++ b/src/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java @@ -163,10 +163,10 @@ public abstract class KylinConfigBase implements Serializable { } public Map<String, String> getReadonlyProperties() { - val substitutor = getSubstitutor(); + val subStitutorTmp = getSubstitutor(); HashMap<String, String> config = Maps.newHashMap(); for (Entry<Object, Object> entry : this.properties.entrySet()) { - config.put((String) entry.getKey(), substitutor.replace((String) entry.getValue())); + config.put((String) entry.getKey(), subStitutorTmp.replace((String) entry.getValue())); } return config; } diff --git a/src/core-common/src/main/java/org/apache/kylin/common/KylinExternalConfigLoader.java b/src/core-common/src/main/java/org/apache/kylin/common/KylinExternalConfigLoader.java index 206539dd7a..880e3ab52f 100644 --- a/src/core-common/src/main/java/org/apache/kylin/common/KylinExternalConfigLoader.java +++ b/src/core-common/src/main/java/org/apache/kylin/common/KylinExternalConfigLoader.java @@ -167,6 +167,7 @@ public class KylinExternalConfigLoader implements ICachedExternalConfigLoader { /** * @see #getPropertyEntries + * @deprecated */ @Override @Deprecated