[ 
https://issues.apache.org/jira/browse/TS-4261?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15183191#comment-15183191
 ] 

ASF GitHub Bot commented on TS-4261:
------------------------------------

Github user jpeach commented on the pull request:

    https://github.com/apache/trafficserver/pull/516#issuecomment-193317236
  
    This looks good. Could you name the new file ``RecRawStats.cc`` or 
``RawStats.cc``?


> Split statistic update logic from process handling.
> ---------------------------------------------------
>
>                 Key: TS-4261
>                 URL: https://issues.apache.org/jira/browse/TS-4261
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: Manager
>            Reporter: Alan M. Carroll
>            Assignee: Alan M. Carroll
>             Fix For: 6.2.0
>
>
> The logic for updating statistics is mixed in the same source file as process 
> management. This creates unnecessary circular dependencies if lower level 
> components use statistics.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to