[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-03-03 Thread gtenev
Github user gtenev commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
@jpeach: sounds like right thing to do, needed a quick fix, will find some 
time to make the API less error-prone. 
@zwoop: sir, yes, sir! 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread zwoop
Github user zwoop commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
We should file an Issue on that improvement.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread jpeach
Github user jpeach commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
Can you look into fixing the APIs so this can't happen again?

> On Feb 28, 2017, at 10:32 AM, Gancho Tenev  
wrote:
> 
> The size of the cache code map does not correspond to the number of cache 
result code values.
> Discrepancy introduced with commit eadc9cf
> 
> You can view, comment on, or merge this pull request online at:
> 
>   https://github.com/apache/trafficserver/pull/1515
> 
> Commit Summary
> 
> Logging cache code map size fix
> File Changes
> 
> M proxy/logging/Log.cc (2)
> Patch Links:
> 
> https://github.com/apache/trafficserver/pull/1515.patch
> https://github.com/apache/trafficserver/pull/1515.diff
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub, or mute the thread.
> 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
clang-analyzer build *successful*! See 
https://ci.trafficserver.apache.org/job/clang-analyzer-github/205/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
Intel CC build *successful*! See 
https://ci.trafficserver.apache.org/job/icc-github/73/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
Linux build *successful*! See 
https://ci.trafficserver.apache.org/job/linux-github/1536/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] trafficserver issue #1515: Logging cache code map size fix

2017-02-28 Thread atsci
Github user atsci commented on the issue:

https://github.com/apache/trafficserver/pull/1515
  
FreeBSD build *successful*! See 
https://ci.trafficserver.apache.org/job/freebsd-github/1640/ for details.
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---