Confuse logic?

[Arvinaa]

This class file: https://github.com/alibaba/nacos/blob/2.2.0/client/src/main/java/com/alibaba/nacos/client/naming/utils/InitUtils.java
In method initNamespaceForNaming, I cant understand always use properties.getProperty(PropertyKeyConst.NAMESPACE);`, only for priint log? there are really confiuse for me, please answer...

tmpNamespace = TemplateUtils.stringEmptyAndThenExecute(tmpNamespace, () -> {
            String namespace = properties.getProperty(PropertyKeyConst.NAMESPACE);
            LogUtils.NAMING_LOGGER.info("initializer namespace from System Property :" + namespace);
            return namespace;
        });
        
        if (StringUtils.isEmpty(tmpNamespace)) {
            tmpNamespace = properties.getProperty(PropertyKeyConst.NAMESPACE);
        }

[lipeishen]

in my opinion, when "tmpNamespace" is empty, "String namespace = properties.getProperty(PropertyKeyConst.NAMESPACE) " is to get the value of "namespace" and assign it to "tmpNamespace" , not just to print log.

[Arvinaa]

in my opinion, when "tmpNamespace" is empty, "String namespace = properties.getProperty(PropertyKeyConst.NAMESPACE) " is to get the value of "namespace" and assign it to "tmpNamespace" , not just to print log.

but two lines code always assign value to variable tpmNamespace, and twice are from properties.getProperty(PropertyKeyConst.NAMESPACE);

[KomachiSion]

The original logic is get NAMESPACE from JVM, if JVM has no value, get from user properties.

and After some refactor for properties, all properties is merged into NacosProperties, it makes the confuse.

[onewe]

@i will resolve it@