[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/752/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/856/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *failed*! See https://ci.trafficserver.apache.org/job/Github-Linux/751/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *failed*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/855/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *failed*! See https://ci.trafficserver.apache.org/job/Github-Linux/750/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *failed*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/854/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-23 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 @jpeach @bryancall add comments and details in commit message ``` Load the NetVCs from open_list to cop_list before next InactivityCop runs. NetHandler will remove NetVC from cop_list

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-09-22 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/771 @oknet would you mind revising the commit subject and descripton? ``` TS-4612: Optimize InactivityCop. Longer description here. Capture the important points from this PR

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-08-27 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 rebased on master --- 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

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-08-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/528/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-08-27 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/632/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-08-17 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/771 Where are we with this? Ready to land? --- 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

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-26 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 @bryancall to review the codes of ClusterHandlerBase.cc ``` 979 thread = eventProcessor.eventthread[ET_CLUSTER][id % num_of_cluster_threads]; 980 if

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 update iocore/cluster/ClusterHandlerBase.cc @bryancall @zwoop but I have no environment to perform cluster system test. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 @bryancall I'm found open_list also used in Cluster. file: iocore/cluster/ClusterHandlerBase.cc ``` 1007 case ClusterHandler::CLCON_CONN_BIND_CLEAR: { 1008

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/372/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/475/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/771 [approve ci] --- 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

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-23 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/771 clang-format is done. --- 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

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-22 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/771 @zwoop Yes, this should be merged after the clang-format. --- 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

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-21 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/470/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-21 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/771 @oknet This fails on the clang-format, can you please fix and push an update? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-21 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/771 Linux build *failed*! See https://ci.trafficserver.apache.org/job/Github-Linux/367/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver issue #771: TS-4612: Proposal: InactivityCop Optimize

2016-07-21 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/771 @bryancall Should we merge this? [approve ci]. --- 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