search

TarsCloud / TarsFramework

Tars Basic service framework
BSD 3-Clause "New" or "Revised" License
198 stars 143 forks source link

Update ServerManager.cpp #141

Closed BeyondWUXF closed 10 months ago

BeyondWUXF commented 10 months ago

修复往不存在的Admin reportNode

ruanshudong commented 10 months ago 
    vector<TC_Endpoint> endPointList = Application::getCommunicator()->getEndpoint4All(_adminObj);

这里是不是应该拿不到之前的代理, 然后从_adminPrxs去掉就好了?

BeyondWUXF commented 10 months ago 
    vector<TC_Endpoint> endPointList = Application::getCommunicator()->getEndpoint4All(_adminObj);

这里是不是应该拿不到之前的代理, 然后从_adminPrxs去掉就好了?

如果从这里判断，每次都得两个循环操作，会不会浪费性能，大部分情况都不需要从_adminPrxs移除

ruanshudong commented 10 months ago

不存在, 十秒才做一次, 不影响. 主要按照现在的做法, 万一网络出点问题就被去掉了, 不合适

BeyondWUXF commented 10 months ago

如果reportNode之前判断一下，如果不在endPointList直接从_adminPrxs移除，这样是不是好一些

ruanshudong commented 10 months ago

也可以的, 你改一下试一试

BeyondWUXF commented 10 months ago

如果reportNode之前判断一下，如果不在endPointList直接从_adminPrxs移除，这样是不是好一些

测试了没问题