[GitHub] trafficserver issue #1509: TS-5106: Create ParentRoundRobin object as Parent...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/1509 Can you rebase this so we can get the Intel build clear? Thanks. --- 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

[GitHub] trafficserver issue #1513: TS-4747: make marking parent proxies down in host...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/1513 Can you rebase this so we can get the Intel build clear? Thanks. --- 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

[GitHub] trafficserver pull request #1485: back port "fix TS-4195: double free when s...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon closed the pull request at: https://github.com/apache/trafficserver/pull/1485 --- 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 issue #1485: back port "fix TS-4195: double free when stop tra...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1485 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/233/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1485: back port "fix TS-4195: double free when stop tra...

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

[GitHub] trafficserver issue #1485: back port "fix TS-4195: double free when stop tra...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1485 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/101/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-03-04 Thread bryancall
Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1465 @zwoop Did you run the tests? --- 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 issue #1485: back port "fix TS-4195: double free when stop tra...

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

[GitHub] trafficserver issue #1276: TS-5105: Do vc->con.setRemote(target) before sock...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/1276 @oknet I think these have been fixed in 6.2.x upstream and you need to rebase. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver issue #1485: back port "fix TS-4195: double free when stop tra...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/1485 Is the double free related to ICC build issues too? --- 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

[GitHub] trafficserver issue #1485: back port "fix TS-4195: double free when stop tra...

2017-03-04 Thread PSUdaemon
Github user PSUdaemon commented on the issue: https://github.com/apache/trafficserver/pull/1485 [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

[GitHub] trafficserver issue #1536: Create RAT builds for all Github PRs

2017-03-04 Thread zwoop
GitHub user zwoop opened an issue: https://github.com/apache/trafficserver/issues/1536 Create RAT builds for all Github PRs This will help us identifying RAT errors / reports before they hit the main repo. --- If your project is set up for it, you can reply to this

[GitHub] trafficserver issue #1535: Cleanup new tests/ tree with RAT exclusions / lic...

2017-03-04 Thread zwoop
GitHub user zwoop opened an issue: https://github.com/apache/trafficserver/issues/1535 Cleanup new tests/ tree with RAT exclusions / licenses Also, we need to include the MIT license as well into LICENSE, and cleanup NOTICES. --- If your project is set up for it, you

[GitHub] trafficserver issue #1465: Add new testing system to the test subdirectory

2017-03-04 Thread zwoop
Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1465 I'm going to land this. There will definitely be RAT report errors on this, but we can address those either with adding licenses when needed, or add to the RAT exclusions (e.g. exclude *.gold

[GitHub] trafficserver pull request #1465: Add new testing system to the test subdire...

2017-03-04 Thread zwoop
Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1465 --- 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 #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/232/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/100/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/231/ for details. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/230/ for details. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/99/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/98/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 clang-analyzer build *failed*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/229/ for details. --- If your project is set up for it, you can reply to this email and

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1533: TS-4976: Regularize example plugin append_transfo...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1533 clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/228/ for details. --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1534 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/97/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1533: TS-4976: Regularize example plugin append_transfo...

2017-03-04 Thread atsci
Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1533 Intel CC build *successful*! See https://ci.trafficserver.apache.org/job/icc-github/96/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1533: TS-4976: Regularize example plugin append_transfo...

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

[GitHub] trafficserver issue #1533: TS-4976: Regularize example plugin append_transfo...

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

[GitHub] trafficserver issue #1534: Use StringView for protocol stack to avoid callin...

2017-03-04 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1534 I checked the compiler output and AFAICT passing a `StringView` by value is the same cost as passing two integers. That is, passing a `StringView` by value has the same performance

[GitHub] trafficserver pull request #1534: Use StringView for protocol stack to avoid...

2017-03-04 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1534 Use StringView for protocol stack to avoid calling strlen on strings with a known length. This is an alternative proposal to #1520. You can merge this pull request into a Git

[GitHub] trafficserver pull request #1533: TS-4976: Regularize example plugin append_...

2017-03-04 Thread SolidWallOfCode
GitHub user SolidWallOfCode opened a pull request: https://github.com/apache/trafficserver/pull/1533 TS-4976: Regularize example plugin append_transform. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] trafficserver issue #1476: Crash in get_client_addr()

2017-03-04 Thread suenway
Github user suenway commented on the issue: https://github.com/apache/trafficserver/issues/1476 This is an automatically generated message. sun...@yahoo-inc.com is no longer with Yahoo! Inc. Your message will not be forwarded. If you require assistance

[GitHub] trafficserver issue #1476: Crash in get_client_addr()

2017-03-04 Thread shinrich
Github user shinrich commented on the issue: https://github.com/apache/trafficserver/issues/1476 Have been adding member variables to NetVConnection and adding assert to HttpSM::state_api_callout. In the cores it seemed that the crash always happened in the first hook called

[GitHub] trafficserver issue #1527: More and slower active connections in 7.1.x

2017-03-04 Thread oknet
Github user oknet commented on the issue: https://github.com/apache/trafficserver/issues/1527 Our product based on 6.0.x and backport some commits from master. With the commit 425b696 , we did not found any vc leaks and performance decrease. Can you show me how to