[GitHub] trafficserver pull request: TS-4261: Split stats API from process ...

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/516 --- 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: Add TSSslContextCreate method

2016-03-19 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/402#issuecomment-197684472 @biilmann The next thing we need here is an [API review](https://cwiki.apache.org/confluence/display/TS/API+Review+Process) posted to the dev@ list. It would also

[GitHub] trafficserver pull request: TS 4271: metrics fail to clear

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/528 --- 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: fix typo

2016-03-19 Thread MatrixYuri
GitHub user MatrixYuri opened a pull request: https://github.com/apache/trafficserver/pull/529 fix typo Hi , I've just found a typo in the document. So here is a fix patch You can merge this pull request into a Git repository by running: $ git pull https://github.com/M

[GitHub] trafficserver pull request: TS-4282: Check for _do_read returning ...

2016-03-19 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/527#issuecomment-197570520 Seems fine, +1. --- 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 f

[GitHub] trafficserver pull request: TS-4282: Check for _do_read returning ...

2016-03-19 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/527 TS-4282: Check for _do_read returning undef in AdminClient.pm You can merge this pull request into a Git repository by running: $ git pull https://github.com/SolidWallOfCode/tra

[GitHub] trafficserver pull request: Ts 4256 fix abort problem with inliner

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/511 --- 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 4256 fix abort problem with inliner

2016-03-19 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/511#issuecomment-198744974 I added a virtual destructor to `HtmlParser` to get rid of compiler warnings. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver pull request: fix typo

2016-03-19 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/529#issuecomment-197926446 Thanks @MatrixYuri --- 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 t

[GitHub] trafficserver pull request: Update header_rewrite.en.rst to add ex...

2016-03-19 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/524#issuecomment-197684969 Thanks @ryandurfey! --- 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 t

[GitHub] trafficserver pull request: Update header_rewrite.en.rst to add ex...

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/524 --- 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 4271: metrics fail to clear

2016-03-19 Thread jpeach
GitHub user jpeach opened a pull request: https://github.com/apache/trafficserver/pull/528 TS 4271: metrics fail to clear Fix failing ``traffic_ctl metric clear`` command. Rename ``RecDataClear`` to ``RecDataZero`` to make it more obvious. You can merge this pull request into a Git

[GitHub] trafficserver pull request: fix typo

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

Re: Github PR Scrub on Tuesday 3/22, 8am PST

2016-03-19 Thread Bryan Call
Here is the URL for the Hangout: https://hangouts.google.com/hangouts/_/tphk5els3jgdhexjbmajwwca7qe -Bryan > On Mar 16, 2016, at 8:31 AM, Leif Hedstrom wrote: > > Hi all, > > we’ll do another Github PR Scrub, on Tuesday

[GitHub] trafficserver pull request: TS-4282: Check for _do_read returning ...

2016-03-19 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/527#issuecomment-197579454 Nope, it's wrong - let me fix it (should use `undefined` rather than `== undef`). Too long since I really worked in Perl. --- If your project is set up f