This is an automated email from the ASF dual-hosted git repository. zwoop pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/9.0.x by this push: new 22b877b Fixes a bug where the nexthop markNextHop method to mark a host down is not called when because the wrapper function was not used. 22b877b is described below commit 22b877b201e72d0b40f5df7d4268a59078d55bb7 Author: John Rushford <jrushf...@apache.org> AuthorDate: Wed Apr 8 16:38:52 2020 +0000 Fixes a bug where the nexthop markNextHop method to mark a host down is not called when because the wrapper function was not used. (cherry picked from commit 90357576c4cbd8c172047955d385b7ba0cd62627) --- proxy/http/HttpTransact.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/proxy/http/HttpTransact.cc b/proxy/http/HttpTransact.cc index cafab42..79d87fe 100644 --- a/proxy/http/HttpTransact.cc +++ b/proxy/http/HttpTransact.cc @@ -3603,7 +3603,7 @@ HttpTransact::handle_response_from_parent(State *s) // us to mark the parent down if (s->current.state == CONNECTION_ERROR) { HTTP_INCREMENT_DYN_STAT(http_total_parent_marked_down_count); - s->parent_params->markParentDown(&s->parent_result, s->txn_conf->parent_fail_threshold, s->txn_conf->parent_retry_time); + markParentDown(s); } // We are done so look for another parent if any next_lookup = find_server_and_update_current_info(s); @@ -3615,7 +3615,7 @@ HttpTransact::handle_response_from_parent(State *s) TxnDebug("http_trans", "[handle_response_from_parent] Error. No more retries."); if (s->current.state == CONNECTION_ERROR) { HTTP_INCREMENT_DYN_STAT(http_total_parent_marked_down_count); - s->parent_params->markParentDown(&s->parent_result, s->txn_conf->parent_fail_threshold, s->txn_conf->parent_retry_time); + markParentDown(s); } s->parent_result.result = PARENT_FAIL; next_lookup = HOST_NONE;