diff --git a/proxy/Milestones.h b/proxy/Milestones.h index b05d60f8ee8..0472aff9512 100644 --- a/proxy/Milestones.h +++ b/proxy/Milestones.h @@ -44,7 +44,7 @@ class TransactionMilestones * Takes two milestones and returns the difference. * @param start The start time * @param end The end time - * @return A double that is the time in seconds + * @return The difference time in milliseconds */ int64_t difference_msec(TSMilestonesType ms_start, TSMilestonesType ms_end) const @@ -55,10 +55,16 @@ class TransactionMilestones return ink_hrtime_to_msec(milestones[ms_end] - milestones[ms_start]); } + /** + * Takes two milestones and returns the difference. + * @param start The start time + * @param end The end time + * @return A double that is the difference time in seconds + */ double - difference(TSMilestonesType ms_start, TSMilestonesType ms_end) const + difference_sec(TSMilestonesType ms_start, TSMilestonesType ms_end) const { - return (double)difference_msec(ms_start, ms_end); + return (double)difference_msec(ms_start, ms_end) / 1000.0; } ink_hrtime diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc index 5803e234f39..b34b9dbe493 100644 --- a/proxy/http/HttpSM.cc +++ b/proxy/http/HttpSM.cc @@ -6949,21 +6949,21 @@ HttpSM::update_stats() "plugin_total: %.3f", sm_id, client_ip, t_state.client_info.src_addr.host_order_port(), url_string, status, unique_id_string, redirection_tries, client_response_body_bytes, fd, t_state.client_info.state, t_state.server_info.state, - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_UA_BEGIN), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_UA_FIRST_READ), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_UA_READ_HEADER_DONE), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_CACHE_OPEN_READ_BEGIN), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_CACHE_OPEN_READ_END), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_DNS_LOOKUP_BEGIN), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_DNS_LOOKUP_END), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_CONNECT), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_FIRST_READ), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_READ_HEADER_DONE), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_CLOSE), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_UA_CLOSE), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_SM_FINISH), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_PLUGIN_ACTIVE), - milestones.difference(TS_MILESTONE_SM_START, TS_MILESTONE_PLUGIN_TOTAL)); + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_UA_BEGIN), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_UA_FIRST_READ), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_UA_READ_HEADER_DONE), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_CACHE_OPEN_READ_BEGIN), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_CACHE_OPEN_READ_END), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_DNS_LOOKUP_BEGIN), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_DNS_LOOKUP_END), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_CONNECT), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_FIRST_READ), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_READ_HEADER_DONE), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_SERVER_CLOSE), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_UA_CLOSE), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_SM_FINISH), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_PLUGIN_ACTIVE), + milestones.difference_sec(TS_MILESTONE_SM_START, TS_MILESTONE_PLUGIN_TOTAL)); } }