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
reply
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
r
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 app
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
reply
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 app
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
reply
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 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 and
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 so
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
reply
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
r
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 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 (net_vc->thread
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/771
@oknet I don't think it is necessary to remove it from the cop_list, since
it is currently not being removed from the list and it is checking the
inactivity timeouts. I would leave it as is
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 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 UnixNetVC
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
reply
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
r
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 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 wishe
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 does
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
r
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 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 app
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
25 matches
Mail list logo