Craig Tiller
613b90ba93
Revert "Buffer HPACK parsing until the end of a header boundary ( #26700 )" ( #26825 )
...
This reverts commit 8bab3e4bf4
.
3 years ago
Craig Tiller
8bab3e4bf4
Buffer HPACK parsing until the end of a header boundary ( #26700 )
...
HTTP2 headers are sent in (potentially) many frames, but all must be
sent sequentially with no traffic intervening.
This was not clear when I wrote the HPACK parser, and still indeed quite
contentious on the HTTP2 mailing lists.
Now that matter is well settled (years ago!) take advantage of the fact
by delaying parsing until all bytes are available.
A future change will leverage this to avoid having to store and verify
partial parse state, completely eliminating indirect calls within the
parser.
3 years ago
Esun Kim
ca945a58e9
Introduced grpc_error_handle ( #25902 )
...
- Define grpc_error_handle
- Replace grpc_error* with grpc_error_handle
4 years ago
Yash Tibrewal
7021b72d1f
Revert Revert Xds Status Notifier ( #25718 )
...
* Revert "Revert "xDS status notifier (#25321 )" (#25702 )"
This reverts commit 3c9f3972e3
.
* Remove connection from map when OnClose is not registered
* Reviewer comments
4 years ago
Yash Tibrewal
3c9f3972e3
Revert "xDS status notifier ( #25321 )" ( #25702 )
...
This reverts commit 81e90432e1
.
4 years ago
Esun Kim
a41a206b8c
Fix readability-const-return-type,readability-delete-null-pointer,readability-simplify-boolean-expr ( #25628 )
4 years ago
Yash Tibrewal
81e90432e1
xDS status notifier ( #25321 )
...
* Serving status notification for xds enabled servers
4 years ago
AJ Heller
0e3a02e903
Revert "Remove the `urgent` argument from iomgr tcp read API ( #25494 )" ( #25592 )
...
This reverts commit a3398f9
. Justification: see b/181367644. tl;dr: assuming urgent==false does not hold in all situations.
4 years ago
AJ Heller
a3398f924c
Remove the `urgent` argument from iomgr tcp read API ( #25494 )
...
The urgent argument is a platform-specific flag that leaked into the (ideally) platform-independent HTTP/2 transport layer. In an effort to clean up the cross-platform API surface, it would be helpful if we can remove this argument from the TCP Read api without losing the performance optimization that was introduced along with it (see #18240 ).
4 years ago
Esun Kim
2ff84ed8c2
C++ cast for void*
4 years ago
Yash Tibrewal
f2fa95b593
Do not cancel RPC if send metadata size if larger than peer's limit
4 years ago
Mark D. Roth
0b61aea1c5
Convert grpc_server to idiomatic C++.
4 years ago
Mark D. Roth
3bb427195e
Revert "Merge pull request #23053 from grpc/revert-22772-gpr_strvec"
...
This reverts commit 4b7452bcdb
, reversing
changes made to 133d0076b4
.
5 years ago
Karthik Ravi Shankar
f46c07f3f8
Revert "Remove gpr_strvec."
5 years ago
Mark D. Roth
c2ef110303
Replace gpr_strvec with absl::StrJoin().
5 years ago
Jan Tattermusch
41ac287f7b
generate build metadata from bazel BUILD
5 years ago
Esun Kim
e52081f903
More pythons to be formatted
5 years ago
Jan Tattermusch
78888d7861
avoid thread local unitialized problem under GRPC_PTHREAD_TLS
5 years ago
Lidi Zheng
c34bef4ca4
Format all Bazel files with buildifier
5 years ago
Vijay Pai
38d87137f8
Remove unused parameter warning (3 of 20)
5 years ago
Vijay Pai
41d89242ad
Remove unused parameter warning (2 of 20)
5 years ago
Esun Kim
feb263ba6d
Add check_attrs & clean-up build.yaml
5 years ago
Yash Tibrewal
8a310e5063
Clang tidy
6 years ago
Yash Tibrewal
4fc02c6bf5
add tests for bad stream IDs
6 years ago
Yash Tibrewal
9f10ab5377
Add gtest dependency to the test instead of library
6 years ago
Yash Tibrewal
e60c43ff3e
Add out of bounds frame tests
6 years ago
Vijay Pai
f05c475f01
Fix Python3 incompatibility
6 years ago
Yash Tibrewal
ea427791d3
Reviewer comments
6 years ago
Yash Tibrewal
597a67a2b5
Covert to GTEST
6 years ago
Yash Tibrewal
7994ea53fc
Add detailed unknown frame type tests
6 years ago
Yash Tibrewal
9345b7e276
s/atleast/at\ least
6 years ago
Soheil Hassas Yeganeh
18b19105f2
Implement TCP_INQ for gRPC in Linux
...
TCP_INQ is a socket option we added to Linux to report pending bytes
on the socket as a control message.
Using TCP_INQ we can accurately decide whether to continue read or not.
Add an urgent parameter, when we do not want to wait for EPOLLIN.
This commit improves the latency of 1 RPC unary (minimal benchmark)
significantly:
Before:
l_50: 61.3584984733
l_90: 94.8328711277
l_99: 126.211351174
l_999: 158.722406029
After:
l_50: 51.3546011488 (-16%)
l_90: 72.3420731581 (-23%)
l_99: 103.280218974 (-18%)
l_999: 130.905689996 (-17%)
6 years ago
Jan Tattermusch
486b1fe320
Fix bad_client_simple_request test.
...
The data of 0xffffffff is actually not illegal, the top bit should be
ingored according to the spec.
6 years ago
ncteisen
7b81ae14a7
clang tidy and clang format
6 years ago
yang-g
e00ca0371a
Promote gpr_test_util to part of grpc_test_util, add a sample to show it works, build.yaml change first
6 years ago
yang-g
e75fc243da
change tests to use the new class instead of grpc_test_init
6 years ago
ncteisen
e35f24da4d
reviewer feedback
6 years ago
Yash Tibrewal
8d47cd4992
Revert "Revert "Fathom tcp changes""
6 years ago
Jan Tattermusch
97ba943f17
Revert "Fathom tcp changes"
6 years ago
Yash Tibrewal
f0397933b0
Fathom TCP level changes. TracedBuffer for keeping track of all buffers
...
to be traced. Adding tests for Fathom and TracedBuffer. A lot more.
Please read PR description.
6 years ago
Vijay Pai
da69355f30
C++ize gpr_thread as grpc_core::Thread, make it 2-phase init (construct/Start)
7 years ago
Vijay Pai
58a62755fc
Remove support for detached threads. All threads must be joined.
7 years ago
Vijay Pai
c745690caf
Internalize gpr_thd except for id and currentid
7 years ago
Noah Eisen
4d20a66685
Run clang fmt
7 years ago
Noah Eisen
be82e64b3d
Autofix c casts to c++ casts
7 years ago
adelez
39307d23c3
Revert "Revert "Define the tests as grpc_cc_test to automatically test against all po…""
7 years ago
adelez
18a356c9fa
Revert "Define the tests as grpc_cc_test to automatically test against all po…"
7 years ago
Adele Zhou
5b30e46854
Define the tests as grpc_cc_test to automatically test against all polling strategies
7 years ago
Yash Tibrewal
83ad85e2fa
Remove unnecessary check and few more comments
7 years ago
Yash Tibrewal
54f6092eff
Force the client to shutdown so that the server write can complete
7 years ago