[GitHub] trafficserver issue #787: TS-4507: Fix SSN and TXN hook ordering.

2016-08-02 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/787 I've reviewed the changes and I don't think they can really be split up. Part of the reason is similar changes in both the HTTP/1 and HTTP/2 classes, which makes the change appear bigg

[GitHub] trafficserver issue #787: TS-4507: Fix SSN and TXN hook ordering.

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

[GitHub] trafficserver issue #787: TS-4507: Fix SSN and TXN hook ordering.

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

[GitHub] trafficserver issue #787: TS-4507: Fix SSN and TXN hook ordering.

2016-07-15 Thread shinrich
Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/787 Pushed new version of the branch that does not include the schedule_event and state_api_callout fixes. Filed TS-4663 and TS-4664 to track those issues. Will try to reproduce those fixes on

[GitHub] trafficserver issue #787: TS-4507: Fix SSN and TXN hook ordering.

2016-07-15 Thread shinrich
Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/787 It would be possible to break out the schedule_event clean up and the state_api_callout into smaller patches. The rest of it really needs to be together.I need to move onto another proje