You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "ASF subversion and git services (JIRA)" <ji...@apache.org> on 2014/04/10 01:47:15 UTC
[jira] [Commented] (TS-2622) optimize the function ink_cluster_time
[ https://issues.apache.org/jira/browse/TS-2622?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13964824#comment-13964824 ]
ASF subversion and git services commented on TS-2622:
-----------------------------------------------------
Commit 8aa3af937548942e8130ccdf1400e043870e3a0e in trafficserver's branch refs/heads/master from [~happy_fish100]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=8aa3af9 ]
TS-2622: optimize the function ink_cluster_time
1. remove useless two global variables
2. use ink_get_hrtime() instead of time(NULL)
3. remove useless CAS to set variable global_time
> optimize the function ink_cluster_time
> --------------------------------------
>
> Key: TS-2622
> URL: https://issues.apache.org/jira/browse/TS-2622
> Project: Traffic Server
> Issue Type: Improvement
> Components: Clustering, HTTP
> Reporter: Yu Qing
> Labels: review
> Fix For: 5.0.0
>
> Attachments: 0001-TS-2622-optimize-the-function-ink_cluster_time.patch
>
>
> in proxy/http/HttpTransact.cc:
> these two global variables are useless, so should be removed.
> volatile ink_time_t global_time;
> volatile int32_t cluster_time_delta;
> in the function ink_cluster_time:
> 1. use ink_get_hrtime() instead of time(NULL)
> 2. remove useless CAS to set variable global_time because global_time NOT be used in other case.
--
This message was sent by Atlassian JIRA
(v6.2#6252)