Commit Graph

4174 Commits

Author SHA1 Message Date
Moiz Haidry da85cec0f2 Add condition to avoid duplicate shutdown 2019-07-19 14:19:23 -07:00
Moiz Haidry f4b1182a10 Addressed review comments 2019-07-19 13:48:35 -07:00
Moiz Haidry 7e2c5845e1 Merge branch 'master' of https://github.com/grpc/grpc into fix_mutex 2019-07-19 12:29:31 -07:00
Moiz Haidry 6d984166bd Porting Aaron Jacob's unit test that detecting the race 2019-07-19 11:45:33 -07:00
Karthik Ravi Shankar f1fc64274a
Merge pull request #19406 from karthikravis/callback-offload-only-not-bg-poller
Run callbacks on same thread if trigerred from background thread
2019-07-19 09:39:07 -07:00
Karthik Ravi Shankar 553eff9cb0 Fix a test failure due to unused variable and formatting 2019-07-18 14:03:19 -07:00
Karthik Ravi Shankar 7461407ad0 Merge branch 'master' into callback-offload-only-not-bg-poller 2019-07-18 14:00:02 -07:00
Richard Belleville 63b4f3d819 Revert "Merge pull request #19673 from markdroth/channel_grpc_init"
This reverts commit 4e21980716, reversing
changes made to 62b8a783fa.
2019-07-18 13:00:24 -07:00
Mark D. Roth 8cc5b8f680 Defer grpc shutdown until after channel destruction. 2019-07-17 16:35:14 -07:00
Alexander Polcyn 3afb0b2635 Fix fallback test breaking on mac bazel 2019-07-15 18:43:01 -07:00
Arjun Roy b1d73a01f1 Removed duplicate static table from hpack table. Removed an or instruction for
every usage of static grpc metadata. Inlined hpack table lookups for static
metadata.

This leads to faster hpack parser creation:
BM_HpackParserInitDestroy 5.32µs ± 1% 0.06µs ± 1%  -98.91% (p=0.000 n=18+19)

And slightly faster parsing:
BM_HpackParserParseHeader<RepresentativeClientInitialMetadata, OnInitialHeader>
456ns ± 1%              435ns ± 1%   -4.74%        (p=0.000 n=18+19)
BM_HpackParserParseHeader<MoreRepresentativeClientInitialMetadata,
OnInitialHeader>
1.06µs ± 2%             1.04µs ± 2%   -1.82%        (p=0.000 n=19+20)

It also yields a slight (0.5 - 1.0 microsecond) reduction in CPU time for
fullstack unary pingpong:
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/512
[polls/iter:3.0001                         ]            23.9µs ± 2%
23.0µs ± 1%  -3.63%          (p=0.002 n=6+6)
BM_UnaryPingPong<TCP, NoOpMutator, NoOpMutator>/0/32768
[polls/iter:3.00015                        ]            35.1µs ± 1%
34.2µs ± 1%  -2.57%          (p=0.036 n=5+3)
BM_UnaryPingPong<MinTCP, NoOpMutator, NoOpMutator>/8/0
[polls/iter:3.00011                        ]            21.7µs ± 3%
21.2µs ± 2%  -2.44%          (p=0.017 n=6+5)
2019-07-15 15:39:31 -07:00
Alexander Polcyn 3958a53bf7 Address review comments; fix sanity 2019-07-11 19:08:27 -07:00
Alexander Polcyn 4999420c7d Add a test client for certain grpclb fallback scenarios 2019-07-11 18:31:21 -07:00
Qiancheng Zhao 444806583d lazy resolving lb policy creation 2019-07-10 15:26:53 -07:00
yang-g 5779dd935a Qualify the error code with StatusCode:: 2019-07-10 13:52:31 -07:00
Julien Boeuf 109edca971 Adding C++ API and implementation for STS credentials:
- marked as experimental.
- also changed the name of a field in the options struct.
2019-07-09 22:28:44 -07:00
Qiancheng Zhao 6de222a6ad refactor response generator in client_lb_end2end_test 2019-07-08 15:37:21 -07:00
Moiz Haidry bf994e48d9 Move grpc async, callback and sync implementation to grpc_impl namespace 2019-07-01 14:05:54 -07:00
Soheil Hassas Yeganeh 39e982a263
Merge pull request #19488 from soheilhy/string-view2
Introduce string_view and use it for gpr_split_host_port
2019-07-01 13:53:16 -04:00
rmstar ce282e7d7e
Merge pull request #19395 from rmstar/c++_on_ios
Fix C++ tests to run on iOS
2019-06-28 11:03:22 -07:00
Soheil Hassas Yeganeh dbf88dd66f Revert "Revert "Introduce string_view and use it for gpr_split_host_port.""
This reverts commit 80c177d4c4.
2019-06-27 20:56:15 -04:00
apolcyn dd5f0fcf48
Merge pull request #19482 from qixuanl1/add-pick-first-unary
add pick_first_unary interop test
2019-06-27 16:22:13 -07:00
Qixuan Li 913acf456b fix minor nits 2019-06-26 22:32:33 +00:00
Qixuan Li 0e85762b67 add pick_first_unary 2019-06-26 13:49:36 -07:00
Soheil Hassas Yeganeh 80c177d4c4
Revert "Introduce string_view and use it for gpr_split_host_port." 2019-06-26 16:22:13 -04:00
Karthik Ravi Shankar 2602bdd3e4 Address review comments 2019-06-26 11:37:54 -07:00
Soheil Hassas Yeganeh f9516b04d5
Merge pull request #19218 from soheilhy/string-view
Introduce string_view and use it for gpr_split_host_port.
2019-06-26 13:15:52 -04:00
Karthik Ravi Shankar 514413de70 Clang formatting errors 2019-06-25 20:14:15 -07:00
Karthik Ravi Shankar c041acb7a7 Fix the unit tests to exercise nesting correctly. 2019-06-25 20:09:06 -07:00
Karthik Ravi Shankar 44160d2b65 Add unit test to check the re-entrancy of callbacks 2019-06-25 15:43:06 -07:00
Moiz Haidry 2a589e253e
Merge pull request #19435 from mhaidrygoog/grpc_to_grpc_impl
Modify codegen to use grpc_impl namespace and other cleanup
2019-06-24 14:07:24 -07:00
Moiz Haidry fbd5957ee8 Sanity and build fixes 2019-06-21 12:28:55 -07:00
Moiz Haidry 63083d4472 Code cleanup 2019-06-21 11:19:34 -07:00
Moiz Haidry 3ce20819cf Modify codegen to use grpc_impl namespace and other cleanups 2019-06-21 10:51:57 -07:00
Soheil Hassas Yeganeh ef0f9bf7ec Introduce string_view and use it for gpr_split_host_port. 2019-06-20 23:19:34 -04:00
Prashant Jaikumar 43628b286f Update googletest version to v1.8.1
Bazel builds of test/cpp/end2end:end2end_test were failing on Mac OS with v1.8.0 due to missing gtest symbols. The issue is not seen in v1.8.1. A WORKSPACE file was added to gtest repo in 1.8.1, so gtest.BUILD can be removed.
2019-06-20 18:17:13 -07:00
Prashant Jaikumar fbfb93c88f Fix C++ tests to run on iOS
- Define grpc_iomgr_run_in_background in iomgr_posix_cfstream.cc
- Use *_IF_SUPPORTED() for death tests
- Move global test init, teardown to SetUpTestCase, TearDownTestCase as GTMGoogleTestRun doesn't run main()
2019-06-20 18:16:20 -07:00
Yash Tibrewal 56a0153f16 Heap allocate the stream object for other benchmark cases too 2019-06-14 14:44:49 -07:00
Yash Tibrewal cceca10a8a Fix data race, heap use-after-free issue in bm_chttp2_transport 2019-06-13 17:05:51 -07:00
Muxi Yan f5551f1138
Revert " Added some Objective C tests and minor bug fixes." 2019-06-12 13:49:17 -07:00
Moiz Haidry 186875337c Sync with master 2019-06-11 16:16:27 -07:00
Karthik Ravi Shankar e4b992ee48 Merge branch 'master' into migrate-client-context 2019-06-10 14:01:28 -07:00
rmstar 340087b3cc
Merge pull request #19022 from rmstar/objcut
Added some Objective C tests and minor bug fixes.
2019-06-10 11:06:18 -07:00
Prashant Jaikumar 2b6e7c4423 Added some Objective C tests and minor bug fixes.
Objective-C tests: metadata, compression, keepalives, channel args.
Stress tests: network flap while streaming call in progress.

Bug fixes:
Stream gzip handling in interop server.
Keep alive, backoff time truncation bug in Obj-C layer.
2019-06-07 10:26:55 -07:00
Mark D. Roth c46eb428cb
Merge pull request #19066 from markdroth/lb_policy_channelz_api_cleanup
Remove channelz from LB policy API.
2019-06-07 08:39:39 -07:00
Mark D. Roth cfb31818ef Remove channelz from LB policy API. 2019-06-07 07:09:39 -07:00
rmstar 7d99c560ec
Merge pull request #19138 from rmstar/cfstreamtestlog
cfstream_test: print HTTP2 stream id of completed RPCs
2019-06-06 13:52:49 -07:00
Lidi Zheng f8b1e6cb44
Merge pull request #19193 from ygj6/dev
some typo errors
2019-06-06 13:03:14 -07:00
rmstar fdbc1b0c86
Merge pull request #19226 from rmstar/cfstream_test_flake
Fix cfstream_test flake
2019-06-06 10:25:31 -07:00
Karthik Ravi Shankar 522ddfe273 Move ClientContext to grpc_impl
ClientContext is another file which needs to be moved from grpc to
grpc_impl for referencing it.
2019-06-05 16:31:39 -07:00