[
https://issues.apache.org/jira/browse/TS-829?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Leif Hedstrom updated TS-829:
-----------------------------
Fix Version/s: (was: 3.3.3)
3.5.0
> socks stats cleanup - some stats are registered, but not used
> -------------------------------------------------------------
>
> Key: TS-829
> URL: https://issues.apache.org/jira/browse/TS-829
> Project: Traffic Server
> Issue Type: Bug
> Components: Network
> Affects Versions: 3.0.0
> Reporter: Bryan Call
> Priority: Minor
> Fix For: 3.5.0
>
>
> From reviewing TS-818 I noticed that the stats that were being double
> resisted are not used. Some cleanup work should be done for the socks stats.
> Stats that are registered, but not used in the code:
> [bcall@snowball traffic.git]$ grep -r proxy.process.socks iocore/net/Net.cc
> "proxy.process.socks.connections_successful",
> "proxy.process.socks.connections_unsuccessful",
> "proxy.process.socks.connections_currently_open",
> These stats are used some tests, so maybe they should be added back into the
> code.
> [bcall@snowball traffic.git]$ grep -rl --binary-files=without-match
> proxy.process.socks.connections_ *
> iocore/net/Net.cc
> mgmt/api/remote/APITestCliRemote.cc
> test/plugin/test-mgmt/test-mgmt.c
> I did however see these stats being used:
> [bcall@snowball traffic.git]$ grep -r SOCKSPROXY_ *
> proxy/SocksProxy.cc:#define SOCKSPROXY_INC_STAT(x) \
> proxy/SocksProxy.cc:
> SOCKSPROXY_INC_STAT(socksproxy_http_connections_stat);
> proxy/SocksProxy.cc:
> SOCKSPROXY_INC_STAT(socksproxy_tunneled_connections_stat);
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
