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

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

Github user bryancall commented on the pull request:

    https://github.com/apache/trafficserver/pull/454#issuecomment-184758654
  
    +1 - Debugging code looks better now


> Cleanup debug logs of Http2ConnectionState
> ------------------------------------------
>
>                 Key: TS-4172
>                 URL: https://issues.apache.org/jira/browse/TS-4172
>             Project: Traffic Server
>          Issue Type: Improvement
>          Components: HTTP/2
>            Reporter: Masaori Koshiba
>            Assignee: Masaori Koshiba
>             Fix For: 6.2.0
>
>
> 1. {{http2_cs}} debug tag is used in {{Http2ClientSession}} and 
> {{Http2ConnectionState}} classes. This sometimes makes me confused when I 
> debug HTTP/2 components.
> It is better to change debug tag of {{Http2ConnectionState}} to 
> {{http2_conn}}.
> 2. Some debug logs has stream id. It is better that other debug logs in 
> {{Http2ConnectionState}} has stream id too.



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

Reply via email to