Github user shukitchan commented on the issue:
https://github.com/apache/trafficserver/pull/1534
ð
Looks good.
Just one question. Should the internal "protocol_contains" functions return
"StringView" as well?
Obviously we should not be changing the return type of
Github user scw00 commented on the issue:
https://github.com/apache/trafficserver/pull/1522
I think, it #never calls handler with read.enabled = 0(or write.enabled).
It means the handler don not want this type of event .It may never handle this
event and just assert!
#1531
---
Github user scw00 commented on the issue:
https://github.com/apache/trafficserver/issues/1531
I think, it never call handler with read.enabled = 0(or write.enabled). It
means the handler don not want this type of event .It may never handle this
event and just assert!
Github user PSUdaemon closed the pull request at:
https://github.com/apache/trafficserver/pull/1539
---
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 user PSUdaemon closed the pull request at:
https://github.com/apache/trafficserver/pull/1509
---
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 user PSUdaemon closed the pull request at:
https://github.com/apache/trafficserver/pull/1276
---
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 user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1509
clang-analyzer build *successful*! See
https://ci.trafficserver.apache.org/job/clang-analyzer-github/237/ for details.
---
If your project is set up for it, you can reply to this email
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/issues/1536
This is fixed.
---
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 user zwoop closed the issue at:
https://github.com/apache/trafficserver/issues/1536
---
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 is
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1276
clang-analyzer build *successful*! See
https://ci.trafficserver.apache.org/job/clang-analyzer-github/236/ for details.
---
If your project is set up for it, you can reply to this email
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1276
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/freebsd-github/1673/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1509
Intel CC build *successful*! See
https://ci.trafficserver.apache.org/job/icc-github/105/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1276
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/linux-github/1569/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1509
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/linux-github/1568/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1509
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/freebsd-github/1672/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1276
Intel CC build *successful*! See
https://ci.trafficserver.apache.org/job/icc-github/104/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user shinrich commented on the issue:
https://github.com/apache/trafficserver/issues/1476
I had two more asserts once I added back in the accept thread. Upon deeper
inspection, the last alloc time and the UA_BEGIN time were off by half a ms, so
I am attributing that to clock
Github user PSUdaemon commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1539#discussion_r104324677
--- Diff: iocore/eventsystem/I_EThread.h ---
@@ -56,7 +56,7 @@ enum ThreadType {
DEDICATED,
};
-extern volatile bool
Github user SolidWallOfCode commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/1539#discussion_r104324657
--- Diff: iocore/eventsystem/I_EThread.h ---
@@ -56,7 +56,7 @@ enum ThreadType {
DEDICATED,
};
-extern volatile bool
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1539
clang-analyzer build *successful*! See
https://ci.trafficserver.apache.org/job/clang-analyzer-github/235/ for details.
---
If your project is set up for it, you can reply to this email
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1539
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/linux-github/1567/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1539
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/freebsd-github/1671/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1539
Intel CC build *successful*! See
https://ci.trafficserver.apache.org/job/icc-github/103/ for details.
---
If your project is set up for it, you can reply to this email and have your
GitHub user PSUdaemon opened a pull request:
https://github.com/apache/trafficserver/pull/1539
Signals fix
This fixed Ubuntu builds but also reverts a commit I just backported
because it was also reverted in master by @zwoop.
You can merge this pull request into a Git repository
Github user dragon512 commented on the issue:
https://github.com/apache/trafficserver/issues/1538
@bryancall Can you provide move info on this. as of Friday this was passing
fine. Could you dump the report details for this test?
---
If your project is set up for it, you can reply to
Github user bryancall closed the issue at:
https://github.com/apache/trafficserver/issues/1443
---
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 user bryancall commented on the issue:
https://github.com/apache/trafficserver/issues/1443
Looks like this should be closed since the PR for it was merged. Please
reopen if need be.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user bryancall opened an issue:
https://github.com/apache/trafficserver/issues/1538
Tests are failing on cache-generation-clear
```
[bcall@homer tests]$ time ./autest.sh --ats-bin /usr/local/bin
Running Test basic:. Passed
Running Test basic-cop:. Passed
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1537
clang-analyzer build *successful*! See
https://ci.trafficserver.apache.org/job/clang-analyzer-github/234/ for details.
---
If your project is set up for it, you can reply to this email
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1537
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/linux-github/1566/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1537
Intel CC build *failed*! See
https://ci.trafficserver.apache.org/job/icc-github/102/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1537
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/freebsd-github/1670/ for details.
---
If your project is set up for it, you can reply to this email and have your
GitHub user SolidWallOfCode opened a pull request:
https://github.com/apache/trafficserver/pull/1537
TS-4976: Regularize example plugin basic_auth.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SolidWallOfCode/trafficserver
Github user SolidWallOfCode closed the pull request at:
https://github.com/apache/trafficserver/pull/1533
---
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
34 matches
Mail list logo
