Yash Tibrewal
e9de13e6ad
Adding test to make sure that http2 transport gets cleaned up ( #25714 )
...
* Add test to make sure that transports get destroyed
* Reviewer comments
4 years ago
Ashitha Santhosh
dbe50923b4
Add getter to grpc_metadata_batch. ( #25395 )
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
Yash Tibrewal
f4e55b85bc
Replace disrespectful term
4 years ago
AJ Heller
880a854f0e
Convert URI parser to C++ #24701
4 years ago
Esun Kim
e0d8c498a6
C++ cast for others
4 years ago
Esun Kim
2ff84ed8c2
C++ cast for void*
4 years ago
Yash Tibrewal
6095b67d6c
Reduce BdpPingNotSentWithoutReceiveSideActivity flakiness
4 years ago
Mark D. Roth
27e1c1c56d
clang-tidy: enable google-readability-braces-around-statements
4 years ago
Yash Tibrewal
b555343dd3
Revert "Revert "Deprecate GRPC_ARG_HTTP2_MIN_SENT_PING_INTERVAL_WITHOUT_DATA_MS""
4 years ago
Esun Kim
e7434d385e
Fix by tidy-modernize-use-override
4 years ago
Esun Kim
6ae8463500
Fix by modernize-make-unique
4 years ago
Yash Tibrewal
a9fd34a81d
Revert "Deprecate GRPC_ARG_HTTP2_MIN_SENT_PING_INTERVAL_WITHOUT_DATA_MS"
4 years ago
Yash Tibrewal
e0c4475db0
Deprecate GRPC_ARG_HTTP2_MIN_SENT_PING_INTERVAL_WITHOUT_DATA_MS and make it non-effective
4 years ago
Yash Tibrewal
23cb073a93
Initiate HTTP/2 writes on BDP pings too
4 years ago
Alexander Polcyn
46d4a27cb6
Add a test which reproduces the original crash that is fixed by https://github.com/grpc/grpc/pull/23984
4 years ago
Yash Tibrewal
4a03ce5a14
Remove grpc_channel_ping from surface API
4 years ago
Yash Tibrewal
09904691e0
Do not send BDP pings if there is no receive side activity.
4 years ago
Yash Tibrewal
7019c2ca4b
Long timeout for too_many_pings_test
4 years ago
Mark D. Roth
bd73844208
Move parse_address library from client_channel to iomgr.
4 years ago
Yash Tibrewal
c62d2b483c
Test renaming
4 years ago
Yash Tibrewal
1618d21796
Add roundrobin test and reviewer comments
4 years ago
Yash Tibrewal
cf0b46c4d6
Reviewer comments
4 years ago
Esun Kim
4c6a6b5a52
Clean up transport tests
4 years ago
Yash Tibrewal
7100ee3bfb
Add keepalive throttling
4 years ago
Yash Tibrewal
2c5291ff71
Plumb absl::Status through connectivity state notifiers
4 years ago
Mark D. Roth
9404f66f70
Replace most uses of gpr_asprintf() with absl calls.
4 years ago
Yash Tibrewal
f8b7fef753
Fix connect deadline issue in settings_timeout_test
4 years ago
Yash Tibrewal
234d64859f
Add some additional comments
5 years ago
Yash Tibrewal
722f1e937a
Reviewer comments
5 years ago
Yash Tibrewal
f41694ad64
Add the missing reset for ping clocks to avoid mistakenly sending GOAWAY
...
frames due to 'too_many_pings'
5 years ago
Yash Tibrewal
376d8689d2
Fix #19094 and #21947 - Fix HTTP status conversion inconsistencies
5 years ago
Brian O'Connor
d38f7cfe5e
Fixing bug with END_STREAM if header has continuations
...
- The HEADER frame should get the END_STREAM flag per the HTTP/2 spec;
the old code put END_STREAM on the last CONTINUATION frame.
- Removing deprecated parameter, is_last_in_stream, from finish_frame();
it has been superseded by the is_end_of_stream member of the framer_state
struct.
- Adding some gRPC frame validation tests to hpack_encoder_test.cc,
and explicting checking that the END_STREAM flag is not on the
CONTINUATION frame.
fixes #21436
5 years ago
Donna Dionne
46439bcb0f
Fixing the settings_timeout_test flake by reducing pollset_work deadline
...
Intermittently (when we run the test 100 times) we see that at times
we are not notified about successful socket connection even within 1
second.
The pollset_work deadline is set to 1 second, so when our connection
timeout is also set to 1 second, we only had 1 poll. And if that poll
didn't let us know about the socket, we didn't get another chance.
Reducing the pollset_work deadline to 100ms, now the test runs faster
and pass consistently:
result for 500 runs:
https://source.cloud.google.com/results/invocations/dd1b38f4-0f28-470b-a78b-b485445e44f2/targets/%2F%2Ftest%2Fcore%2Ftransport%2Fchttp2:settings_timeout_test/tests
5 years ago
Jan Tattermusch
cbb0ef2717
try marking some offenders as flaky
5 years ago
yang-g
5bc57b64fe
InitGoogleTest before grpc_init
5 years ago
Esun Kim
74d2fb842e
Timeout for shutdown!
5 years ago
Esun Kim
9ae81e1d3f
Make test code use TestEnvironment
5 years ago
Esun Kim
3a519a0b64
Replaced grpc_core New & Delete with C++ new & delete
5 years ago
Lidi Zheng
c34bef4ca4
Format all Bazel files with buildifier
5 years ago
Vijay Pai
2477b966e2
Remove unused-parameter warnings, round 2 (17 of 19)
5 years ago
Vijay Pai
17c8c4c4f2
Remove unused parameter warning (13 of 20)
5 years ago
Vijay Pai
c3faf1c3b8
Remove unused parameter warning (12 of 20)
5 years ago
Mark D. Roth
5e9404e41a
Fix polymorphism for UniquePtr<> and OrphanablePtr<>.
5 years ago
Mark D. Roth
dbad2db848
Immediately orphan watcher if state is SHUTDOWN when it is added.
5 years ago
Mark D. Roth
cb67811c47
Revert "Merge pull request #20407 from markdroth/transport_connectivity_state_watcher_revert"
...
This reverts commit 48550de866
, reversing
changes made to 24b529e408
.
5 years ago
Mark D. Roth
33f139b6e6
Revert "Merge pull request #20255 from markdroth/transport_connectivity_state_watcher"
...
This reverts commit 0f5a111aad
, reversing
changes made to 1276a8f628
.
5 years ago
Jan Tattermusch
8e79705d29
disable fuzzers on windows
5 years ago
Jan Tattermusch
b8a32c0cff
temporarily disable failing tests on windows
5 years ago