[GitHub] trafficserver pull request #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-07 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/783 --- 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 issue #783: TS-3816 : Replace ptr_len_cmp with memcmp

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

[GitHub] trafficserver issue #783: TS-3816 : Replace ptr_len_cmp with memcmp

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

[GitHub] trafficserver issue #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-07 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/783 [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 #783: TS-3816 : Replace ptr_len_cmp with memcmp

2016-07-07 Thread jpeach
Github user jpeach commented on the issue: https://github.com/apache/trafficserver/pull/783 👍 --- 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

[GitHub] trafficserver pull request #789: TS-4100: Remove remaining stats processor c...

2016-07-07 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/789 --- 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 issue #789: TS-4100: Remove remaining stats processor code.

2016-07-07 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/789 :+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 feature enabled and wishes so, or if the f

[GitHub] trafficserver issue #789: TS-4100: Remove remaining stats processor code.

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

[GitHub] trafficserver issue #789: TS-4100: Remove remaining stats processor code.

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

[GitHub] trafficserver pull request #789: TS-4100: Remove remaining stats processor c...

2016-07-07 Thread jpeach
GitHub user jpeach opened a pull request: https://github.com/apache/trafficserver/pull/789 TS-4100: Remove remaining stats processor code. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jpeach/trafficserver fix/4100-2 Alternati

[GitHub] trafficserver issue #788: TS-4635: Fix RefCountCacheSerializer file descript...

2016-07-07 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/788 :+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 feature enabled and wishes so, or if the f

[GitHub] trafficserver issue #788: TS-4635: Fix RefCountCacheSerializer file descript...

2016-07-07 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/788 Hmmm, this code looks overly complicated already to begin with :-/. Why is RefCountCacheSerializer a template class? It's tightly coupled with HostDB it seems, and is only used to contain HostDB