[GitHub] trafficserver pull request: TS-4053: Add regression tests for hit ...

2015-12-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/364 --- 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: [VOTE] Release Apache Traffic Server 6.0.1 (RC0)

2015-12-06 Thread Bryan Call
The vote didn’t pass. There was one +1 binding and one -1 non-binding votes. I create another release candidate with a fix for TS-4055 in it. -Bryan > On Dec 2, 2015, at 3:54 PM, Bryan Call wrote: > > I've prepared a release for 6.0.1 (RC0) which is the next bug fix version of > Apache Tra

[GitHub] trafficserver pull request: TS-4018: Use HPACK huffman encoding al...

2015-12-06 Thread bryancall
Github user bryancall commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/362#discussion_r46776397 --- Diff: proxy/http2/HPACK.cc --- @@ -332,18 +332,41 @@ int64_t encode_string(uint8_t *buf_start, const uint8_t *buf_end, const char *value, si

[GitHub] trafficserver pull request: TS-4018: Use HPACK huffman encoding al...

2015-12-06 Thread bryancall
Github user bryancall commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/362#discussion_r46776556 --- Diff: proxy/http2/HPACK.cc --- @@ -332,18 +332,41 @@ int64_t encode_string(uint8_t *buf_start, const uint8_t *buf_end, const char *value, si

[GitHub] trafficserver pull request: TS-3418: Second hash ring for consiste...

2015-12-06 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/359#issuecomment-162380212 Hi John. First, let's clean up the commit history on this branch. - please squash the "clang format" and "updated comments" commits into their respective p

[GitHub] trafficserver pull request: TS-4018: Use HPACK huffman encoding al...

2015-12-06 Thread maskit
Github user maskit commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/362#discussion_r46780362 --- Diff: proxy/http2/HPACK.cc --- @@ -332,18 +332,41 @@ int64_t encode_string(uint8_t *buf_start, const uint8_t *buf_end, const char *value, size_

[GitHub] trafficserver pull request: TS-4018: Use HPACK huffman encoding al...

2015-12-06 Thread masaori335
Github user masaori335 commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/362#discussion_r46780770 --- Diff: proxy/http2/HPACK.cc --- @@ -332,18 +332,41 @@ int64_t encode_string(uint8_t *buf_start, const uint8_t *buf_end, const char *value, s

[GitHub] trafficserver pull request: TS-4018: Use HPACK huffman encoding al...

2015-12-06 Thread maskit
Github user maskit commented on the pull request: https://github.com/apache/trafficserver/pull/362#issuecomment-162387824 I fixed the compile error, and squashed the commits. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] trafficserver pull request: TS-4042: Add feature to buffer request...

2015-12-06 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/351#issuecomment-162403177 @sudheerv , the chunked encoding case is one that I don't think any browser actually does, have you ever seen a browser make a request w/ chunked encoding if so I'l

[GitHub] trafficserver pull request: TS-4042: Add feature to buffer request...

2015-12-06 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/351#issuecomment-162415337 @sudheerv, I've asked @zizhong to help in addressing the transfer-encoding chunked case, we should have an update soon. --- If your project is set up for it, you c