开发者社区 > 云原生 > 正文

关于nacos-client中的HostReactor的一点疑问

最近在学习Nacos相关的源码中,在com.alibaba.nacos:nacos-client:1.4.0中, com.alibaba.nacos.client.naming.core.HostReactor.getServiceInfo:261行

if (null == serviceObj) { serviceObj = new ServiceInfo(serviceName, clusters);

        serviceInfoMap.put(serviceObj.getKey(), serviceObj);
        
        updatingMap.put(serviceName, new Object());
        updateServiceNow(serviceName, clusters);
        updatingMap.remove(serviceName);
        
    } else if (updatingMap.containsKey(serviceName)) {

当有新的服务注册的时候,我发现此处 serviceInfoMap 若不存在该新服务,仍然会将服务信息put到 serviceInfoMap 中去,这将导致 HostReacot.processServiceJson(json) 中 oldService != null 的判断恒为 真

public ServiceInfo processServiceJson(String json) { ServiceInfo serviceInfo = JacksonUtils.toObj(json, ServiceInfo.class); ServiceInfo oldService = serviceInfoMap.get(serviceInfo.getKey()); if (serviceInfo.getHosts() == null || !serviceInfo.validate()) { //empty or error push, just ignore return oldService; }

    boolean changed = false;
    
    if (oldService != null) {
        
        if (oldService.getLastRefTime() > serviceInfo.getLastRefTime()) {
            NAMING_LOGGER.warn("out of date data received, old-t: " + oldService.getLastRefTime() + ", new-t: "
                    + serviceInfo.getLastRefTime());
        }
        
        serviceInfoMap.put(serviceInfo.getKey(), serviceInfo);

当然,我并没有完全读懂整个代码的逻辑,但是我希望有人能给我解释一下。不胜感激

原提问者GitHub用户VincentCh01

展开
收起
学习娃 2023-05-22 16:40:43 114 0
1 条回答
写回答
取消 提交回答
  • 这里的确会是恒为真。这里判断if(olsService != null)应该是为了代码的严谨。虽然之前的流程已经保证了oldService一定存在,但是代码后续的迭代过程中,之前的逻辑可能会有变动,这里判断oldService是比较保险的做法。

    原回答者GitHub用户horizonzy

    2023-05-23 09:58:13
    赞同 展开评论 打赏
问答分类:
问答地址:

阿里云拥有国内全面的云原生产品技术以及大规模的云原生应用实践,通过全面容器化、核心技术互联网化、应用 Serverless 化三大范式,助力制造业企业高效上云,实现系统稳定、应用敏捷智能。拥抱云原生,让创新无处不在。

相关电子书

更多
低代码开发师(初级)实战教程 立即下载
冬季实战营第三期:MySQL数据库进阶实战 立即下载
阿里巴巴DevOps 最佳实践手册 立即下载