[GitHub] trafficserver pull request: Add new limit rate example plugin

2016-05-11 Thread yatsukhnenko
Github user yatsukhnenko commented on the pull request: https://github.com/apache/trafficserver/pull/615#issuecomment-218671191 There is no Jira for this PR and I can't create new issue :( --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] trafficserver pull request: TS-4435: Enable compilation with opens...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/630#issuecomment-218635305 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/21/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4435: Enable compilation with opens...

2016-05-11 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/630 TS-4435: Enable compilation with openssl 1.1 Openssl 1.1 removes the locking and thread id callbacks. It deduces system you are on and uses the appropriate threading interface. Ope

[GitHub] trafficserver pull request: Add new limit rate example plugin

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/615#issuecomment-218620870 Also, would you mind adding the Jira number to the Subject line to this commit / PR? It helps us associate the PR with the correct Jira. --- If your project is set

[GitHub] trafficserver pull request: TS-4309: Simplify read/write loops to ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/629#issuecomment-218621761 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/20/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4309: Simplify read/write loops to ...

2016-05-11 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/629 TS-4309: Simplify read/write loops to address upload/download speed p… …roblems. May also help with ASAN crash reported in TS-4424. You can merge this pull request into a Git re

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread shinrich
Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/627 --- 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 featur

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218620277 Wasn't working on the latest greatest when creating this branch. Closing this PR and opening a new one on a new branch. --- If your project is set up for it, y

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/628#issuecomment-218617861 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/19/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/628#issuecomment-218616131 Ok, I fixed the build issue I think (out of memory), kicking off a new build. [approve ci] --- If your project is set up for it, you can reply to this email and ha

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/628#issuecomment-218615012 Linux (CentOS7) build failed! Details on https://ci.trafficserver.apache.org/job/Github-Linux/18/ --- If your project is set up for it, you can reply to thi

[GitHub] trafficserver pull request: TS-4072 Diagnostic log rolling races

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/568#issuecomment-218614205 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/17/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/628#issuecomment-218614105 Sigh, the build terminated with the Jenkins agent taking a dive it seems... Try again. [approve ci] --- If your project is set up for it, you can reply to this ema

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/628#issuecomment-218613456 Linux (CentOS7) build failed! Details on https://ci.trafficserver.apache.org/job/Github-Linux/16/ --- If your project is set up for it, you can reply to thi

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218612986 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/15/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/628 TS-4032: Enable command line based message passing to plugins. This is the second round after PR-343. I made the RPC unpacking changes HRP recommended. Why they didn't compile before

[GitHub] trafficserver pull request: TS-4032: Enable command line based mes...

2016-05-11 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/343#issuecomment-218612497 I changed the RPC unpacking to use the standard mechanism. I don't know why it didn't compile before and I don't the energy to figure it out. --- If your

[GitHub] trafficserver pull request: TS-4072 Diagnostic log rolling races

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

[GitHub] trafficserver pull request: Ts 4309

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

[GitHub] trafficserver pull request: TS-3245 : Changes optind = 1 to optind...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/571#issuecomment-218591913 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/14/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-3245 : Changes optind = 1 to optind...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/571#issuecomment-218588544 [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 fea

[GitHub] trafficserver pull request: TS-3245 : Changes optind = 1 to optind...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/571#issuecomment-218586966 [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 fe

[GitHub] trafficserver pull request: TS-3607: Integrate ats_pagespeed into ...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/210#issuecomment-218585959 Sure either way is fine. --- 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 h

[GitHub] trafficserver pull request: TS-4413: Fix potential consistent hash...

2016-05-11 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/613 --- 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-3123 Make proxy.config.http.transac...

2016-05-11 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/620 --- 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 i

[GitHub] trafficserver pull request: TS-4410 : Fixed compiler warning on i3...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/611#issuecomment-218582907 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/13/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218582035 Thanks @oknet --- 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-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/625 --- 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: Removed remnants of h2-14 identifier

2016-05-11 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/228 --- 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 i

[GitHub] trafficserver pull request: Removed remnants of h2-14 identifier

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/228#issuecomment-218581467 I'm going to close this, it doesn't look right to me, not sure if we'll see a new PR soon? --- If your project is set up for it, you can reply to this email and ha

[GitHub] trafficserver pull request: TS-4425: Make Ptr pointer value pri...

2016-05-11 Thread jpeach
Github user jpeach closed the pull request at: https://github.com/apache/trafficserver/pull/565 --- 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-4425: Make Ptr pointer value pri...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/565#issuecomment-218580928 Merged. --- 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] trafficserver pull request: TS-4425: Make Ptr pointer value pri...

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

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/593#issuecomment-218578787 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/12/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4410 : Fixed compiler warning on i3...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/611#issuecomment-218578478 Try another build: [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

[GitHub] trafficserver pull request: TS-4410 : Fixed compiler warning on i3...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/611#issuecomment-218577287 Linux (CentOS7) build failed! Details on https://ci.trafficserver.apache.org/job/Github-Linux/11/ --- If your project is set up for it, you can reply to thi

[GitHub] trafficserver pull request: TS-4156: remove the traffic_sac, stand...

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

[GitHub] trafficserver pull request: TS-4410 : Fixed compiler warning on i3...

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

[GitHub] trafficserver pull request: TS-4413: Fix potential consistent hash...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/613#issuecomment-218569956 Linux (CentOS7) build finished successfully. Details on https://ci.trafficserver.apache.org/job/Github-Linux/10/ --- If your project is set up for it, you c

[GitHub] trafficserver pull request: TS-4413: Fix potential consistent hash...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/613#issuecomment-218567588 Started Linux (CentOS7) build. --- 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] trafficserver pull request: TS-4413: Fix potential consistent hash...

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

plugin promotion proposal

2016-05-11 Thread Miles Libbey
At the ATS Summit in Vancouver, we had a brief discussion about each the experimental plugins for the upcoming 7.0 release. In the room, the consensus was: - Move to stable (eg, used, and unlikely to be breaking compatibility in the short term):authproxybackground_fetchesigeneratorregex_revalida

[GitHub] trafficserver pull request: [TS-4103] mocks

2016-05-11 Thread myraid
Github user myraid commented on the pull request: https://github.com/apache/trafficserver/pull/408#issuecomment-218546235 I have to write an example test using the mocks. Let me use webp_transform and write a test. --- If your project is set up for it, you can reply to this email an

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218545767 I probably forgot to run clang format on it. Will clang-format and push again. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218546571 Yep, no problem. I just used this PR to test the new automated Jenkins builds :-). --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218544819 Build finished successfully. No test results found. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218541919 This build failed, please check Jenkins for details. No test results found. --- If your project is set up for it, you can reply to this email and have your re

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218541607 Build triggered. sha1 is merged. --- 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 doe

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218541604 Build triggered. sha1 is merged. --- 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 doe

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

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

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218538166 Hmmm, so, this build failed with wrong clang-format, I'll investigate how to get it to produce proper errors. --- If your project is set up for it, you can reply t

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218537713 Build finished. --- 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

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218537364 Build triggered. sha1 is merged. --- 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 doe

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218537407 Build started sha1 is merged. --- 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] trafficserver pull request: Ts 4309

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

[GitHub] trafficserver pull request: TS-3607: Integrate ats_pagespeed into ...

2016-05-11 Thread oschaaf
Github user oschaaf commented on the pull request: https://github.com/apache/trafficserver/pull/210#issuecomment-218528146 The patch looks good to me, but @jpeach had an improvement in mind. @jpeach would you be OK with committing this and doing https://github.com/apache/trafficserver

[GitHub] trafficserver pull request: TS-3123 Make proxy.config.http.transac...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/620#issuecomment-218525931 Build finished. --- 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

[GitHub] trafficserver pull request: TS-3123 Make proxy.config.http.transac...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/620#issuecomment-218525626 Build started sha1 is merged. --- 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] trafficserver pull request: Add new limit rate example plugin

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/615#issuecomment-218525606 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: AARCH64 fixes tested on Cavium Thunder...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/502#issuecomment-218525666 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: Converted Apache::TS::AdminClient to u...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/35#issuecomment-218525721 Can one of the admins verify this patch? --- 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 proj

[GitHub] trafficserver pull request: TS-4072 Diagnostic log rolling races

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/568#issuecomment-218525646 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4291 Adds log field "pqnhl" which i...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/588#issuecomment-218525638 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: Removed remnants of h2-14 identifier

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/228#issuecomment-218525695 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4410 : Fixed compiler warning on i3...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/611#issuecomment-218525622 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: UUID Proposal

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/199#issuecomment-218525707 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4425: Make Ptr pointer value pri...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/565#issuecomment-218525655 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-3607: Integrate ats_pagespeed into ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/210#issuecomment-218525702 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-3792: Crash with non-existant or mi...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/262#issuecomment-218525691 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-3978: Allow empty document caching ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/310#issuecomment-218525685 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: Added (Estimated) Round Trip Time acce...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/123#issuecomment-218525716 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: [TS-4103] mocks

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/408#issuecomment-218525676 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4274 Use the right hostdb levels as...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/536#issuecomment-218525659 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-3245 : Changes optind = 1 to optind...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/571#issuecomment-218525641 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: [TS-4379] Add log field for Server Con...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/598#issuecomment-218525630 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: [TS-4408] Make volume.config not order...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/618#issuecomment-218525599 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4413: Fix potential consistent hash...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/613#issuecomment-218525617 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: C++ API WebSocket example

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/624#issuecomment-218525564 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: Ts 4309

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/627#issuecomment-218525551 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-3123 Make proxy.config.http.transac...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/620#issuecomment-218525594 Build triggered. sha1 is merged. --- 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 doe

[GitHub] trafficserver pull request: TS-4432: TSVConnFdCreate, double free ...

2016-05-11 Thread atsci
Github user atsci commented on the pull request: https://github.com/apache/trafficserver/pull/625#issuecomment-218525556 Can one of the admins verify this patch? --- 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 pro

[GitHub] trafficserver pull request: TS-4433: Did not consider NULL return ...

2016-05-11 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/626#issuecomment-218517477 Thanks @oknet. I split the error condition so we could log separate debug messages. --- If your project is set up for it, you can reply to this email and have you

[GitHub] trafficserver pull request: TS-3123 Make proxy.config.http.transac...

2016-05-11 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/620#issuecomment-218517598 Testing a little, so be nice. [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 p

[GitHub] trafficserver pull request: TS4433: Did not consider NULL return f...

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