GithubStar/workflow
1
0
Fork 0
mirror of https://github.com/sogou/workflow.git synced 2026-10-26 01:30:47 +08:00
Code Issues Packages Projects Releases Wiki Activity

avoid Upstream check_and_get() with addr==NULL in consistent hash policy

Browse Source
This commit is contained in:
holmes1412
2021-05-27 23:36:49 +08:00
parent 5740f9de73
commit dc5c429534
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/nameservice/UpstreamPolicies.cc
+7 -3
View File
@@ -206,15 +206,16 @@ bool UPSGroupPolicy::select(const ParsedURI& uri, WFNSTracing *tracing,
}
/*
* flag true : guarantee addr != NULL, and please return an available one.
* if no available addr, return NULL.
* false: addr may be NULL, means addr maybe useful but want one any way.
* flag true : return an available one. If not exists, return NULL.
* false: means addr maybe group-alive.
* If addr is not available, get one from addr->group.
*/
inline const EndpointAddress *UPSGroupPolicy::check_and_get(const EndpointAddress *addr,
bool flag,
WFNSTracing *tracing)
{
UPSAddrParams *params = static_cast<UPSAddrParams *>(addr->params);
if (flag == true) // && addr->fail_count >= addr->params->max_fails
{
if (params->group_id == -1)
@@ -440,6 +441,9 @@ const EndpointAddress *UPSGroupPolicy::consistent_hash_with_group(unsigned int h
}
}
if (!addr)
return NULL;
return this->check_and_get(addr, false, NULL);
}