Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/632
---
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 a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63748572
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size)
}
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63741679
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size)
}
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63740460
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size)
}
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63740371
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size)
}
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63739375
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -906,74 +957,140 @@
Http2ConnectionState::update_initial_rwnd(Http2WindowSize new_size)
}
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-219591587
The new commit passed the build tests on the CI
- https://ci.trafficserver.apache.org/view/github/job/Github-Linux/80/
- https://ci.trafficserver.ap
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63255750
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -238,22 +241,29 @@ rcv_headers_frame(Http2ConnectionState &cstate, const
Http2Frame &frame)
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63251327
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -238,22 +241,29 @@ rcv_headers_frame(Http2ConnectionState &cstate, const
Http2Frame &frame)
Github user bryancall commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63248097
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -238,22 +241,29 @@ rcv_headers_frame(Http2ConnectionState &cstate, const
Http2Frame &frame)
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63215167
--- Diff: proxy/http2/Http2DependencyTree.h ---
@@ -0,0 +1,296 @@
+/** @file
+
+ HTTP/2 Dependency Tree
+
+ The original idea o
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63212972
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1976,6 +1976,8 @@ static const RecordElement RecordsConfig[] =
//
{RECT_CONFIG, "pr
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63081400
--- Diff: proxy/http2/Http2DependencyTree.h ---
@@ -0,0 +1,296 @@
+/** @file
+
+ HTTP/2 Dependency Tree
+
+ The original idea of Str
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63078730
--- Diff: proxy/http2/Http2ConnectionState.cc ---
@@ -756,6 +773,15 @@ Http2ConnectionState::main_event_handler(int event,
void *edata)
return 0
Github user masaori335 commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63078637
--- Diff: proxy/http2/HTTP2.h ---
@@ -60,6 +60,10 @@ const uint32_t HTTP2_MAX_FRAME_SIZE = 16384;
const uint32_t HTTP2_HEADER_TABLE_SIZE = 4096;
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63075928
--- Diff: proxy/http2/HTTP2.h ---
@@ -60,6 +60,10 @@ const uint32_t HTTP2_MAX_FRAME_SIZE = 16384;
const uint32_t HTTP2_HEADER_TABLE_SIZE = 4096;
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63075422
--- Diff: proxy/http2/HTTP2.cc ---
@@ -681,6 +686,7 @@ Http2::init()
REC_EstablishStaticConfigInt32U(max_concurrent_streams_in,
"proxy.config.http
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63075156
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1976,6 +1976,8 @@ static const RecordElement RecordsConfig[] =
//
{RECT_CONFIG, "proxy.
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/632#discussion_r63074614
--- Diff: mgmt/RecordsConfig.cc ---
@@ -1976,6 +1976,8 @@ static const RecordElement RecordsConfig[] =
//
{RECT_CONFIG, "proxy.c
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-218847465
This is running on docs.trafficserver right now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user masaori335 commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-218822758
This is second patch after the #525 and TS-4295.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user atsci commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-218811459
FreeBSD v10 build finished successfully. Details on
https://ci.trafficserver.apache.org/job/Github-FreeBSD/13/
---
If your project is set up for it, you can
Github user atsci commented on the pull request:
https://github.com/apache/trafficserver/pull/632#issuecomment-21881
Linux (CentOS7) build finished successfully. Details on
https://ci.trafficserver.apache.org/job/Github-Linux/30/
---
If your project is set up for it, you c
GitHub user masaori335 opened a pull request:
https://github.com/apache/trafficserver/pull/632
TS-3535: Experimental Support of HTTP/2 Stream Priority feature
This is an experimental support of HTTP/2 Stream Priority feature.
- Add a option to enable this feature ( disabled
24 matches
Mail list logo