[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/646 --- 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

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-19 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/646#issuecomment-220376259 . --- 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 feat

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-19 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/646#issuecomment-220368272 I think we should squash these three commits to one, right? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-18 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/646#issuecomment-220056278 Looks reasonable to me. The isEosRcvd was added as part of TS-3714. Much of that buffer saving logic has already been removed because changes in handshake buff

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-18 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/646#issuecomment-220007659 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-18 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/646#issuecomment-220007429 Can one of the admins verify this patch? Only approve PRs which have been reviewed. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4455: remove isEosRcvd() and eosRcv...

2016-05-18 Thread oknet
GitHub user oknet opened a pull request: https://github.com/apache/trafficserver/pull/646 TS-4455: remove isEosRcvd() and eosRcvd You can merge this pull request into a Git repository by running: $ git pull https://github.com/oknet/trafficserver patch-12 Alternatively you can