d970489db2
use no_windows for some C++ tests
2019-09-30 13:27:57 +02:00
24b529e408
Merge pull request #20343 from jtattermusch/bazel_tests_uses_polling
...
Set "uses_polling" arg for bazel tests
2019-09-30 08:13:31 +02:00
6ced125bb3
xds failover locality handling
2019-09-26 17:20:44 -07:00
5d7e4bb301
Stop using deprecated gtest macros
2019-09-24 14:56:41 -07:00
6ebc1c616f
set uses_polling=False for some bazel tests
2019-09-24 09:50:00 +02:00
b0e428c28a
Merge pull request #20245 from markdroth/rr_connectivity_state
...
Change RR to consider a subchannel in TF if it has failed since it was READY.
2019-09-19 08:37:24 -07:00
b1a56e0527
Merge pull request #20270 from AspirinSJL/mv_proto_2
...
Move xds protos to testing/
2019-09-18 20:07:05 -07:00
40c0519ad0
Move xds protos to testing/
2019-09-18 10:20:58 -07:00
b4277d191c
Merge pull request #20269 from AspirinSJL/tune
...
Tune xds test
2019-09-16 16:10:53 -07:00
7be5d098cf
Tune xds test
2019-09-16 13:42:12 -07:00
01eeec72c6
Change RR to consider a subchannel in TF if it has failed since it was READY.
2019-09-16 09:41:01 -07:00
939830aa15
Merge pull request #20030 from karthikravis/move-method-handler
...
Make MethodHandler and related interfaces generic
2019-09-10 21:45:08 -07:00
9f48cb0f07
Merge pull request #19907 from soheilhy/call-cycle-clock
...
Use cycle clock instead of clock monotonic to measure call latency.
2019-09-10 14:36:27 -04:00
f3cd878e0b
Merge pull request #20188 from yashykt/creds_interception
...
Allow call credentials interception at PRE_SEND_INITIAL_METADATA
2019-09-09 18:42:11 -07:00
aa84ceb38e
Add another test for when the credentials were previously set
2019-09-09 15:59:59 -07:00
c9376b4e0b
Use cycle clock instead of clock monotonic to measure call latency.
...
This removes two more getttime syscalls from the hot path, when cycle
clock is enabled.
2019-09-09 10:37:48 -04:00
361a902262
Reviewer comments
2019-09-06 14:20:39 -07:00
0b06676c9e
hpack encoder optimizations.
...
Removed some cycles and branches from hpack_enc for CH2.
Specifically:
1. Pushed certain metadata key/value length checks to
prepare_application_metadata() in src/core/lib/surface/call.cc.
This means that rather than check all key/val lengths for all metadata, we only
do so for custom added user metadata. Inside CH2, we change the length checks to
debug checks so we can catch if core/filter metadata fails to pass the check.
2. Changed various asserts to debug asserts when able.
3. Refactored some of the header emission code to remove duplicated code.
4. Un-inlined some logging methods.
This results in somewhat faster hpack_encoder performance:
BM_HpackEncoderInitDestroy
222ns ± 0% 221ns ± 0% -0.29% (p=0.000 n=34+34)
BM_HpackEncoderEncodeDeadline
[framing_bytes/iter:9 header_bytes/iter:6 ] 135ns ± 1%
124ns ± 0% -8.05% (p=0.000 n=39+38)
BM_HpackEncoderEncodeHeader<EmptyBatch>/0/16384
[framing_bytes/iter:9 header_bytes/iter:0 ] 34.2ns ± 0%
34.2ns ± 0% -0.01% (p=0.014 n=34+38)
BM_HpackEncoderEncodeHeader<EmptyBatch>/1/16384
[framing_bytes/iter:9 header_bytes/iter:0 ] 34.2ns ± 0%
34.2ns ± 0% -0.04% (p=0.004 n=34+37)
BM_HpackEncoderEncodeHeader<SingleStaticElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.5ns ± 0%
45.9ns ± 0% -3.28% (p=0.000 n=28+38)
BM_HpackEncoderEncodeHeader<SingleInternedKeyElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:6 ] 77.0ns ± 1%
68.3ns ± 1% -11.33% (p=0.000 n=39+40)
BM_HpackEncoderEncodeHeader<SingleInternedElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.7ns ± 1%
45.5ns ± 0% -4.63% (p=0.000 n=39+33)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<1, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.2ns ± 0%
45.3ns ± 0% -3.96% (p=0.000 n=33+34)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<3, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.7ns ± 0%
45.6ns ± 0% -4.54% (p=0.000 n=38+40)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<10, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.7ns ± 0%
45.5ns ± 0% -4.63% (p=0.000 n=39+32)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<31, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.8ns ± 0%
45.6ns ± 1% -4.59% (p=0.000 n=38+39)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<100, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.8ns ± 0%
45.5ns ± 0% -4.64% (p=0.000 n=39+36)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<1, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.3ns ± 0%
45.3ns ± 0% -4.09% (p=0.000 n=38+36)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<3, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.8ns ± 1%
45.6ns ± 0% -4.71% (p=0.000 n=37+40)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<10, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.7ns ± 0%
45.5ns ± 0% -4.66% (p=0.000 n=39+32)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<31, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.8ns ± 1%
45.6ns ± 1% -4.62% (p=0.000 n=37+39)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<100, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.7ns ± 0%
45.5ns ± 0% -4.67% (p=0.000 n=38+32)
BM_HpackEncoderEncodeHeader<SingleNonInternedElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:9 ] 80.5ns ± 1%
74.7ns ± 0% -7.16% (p=0.000 n=38+35)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<1, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:12 ] 105ns ± 1%
99ns ± 0% -5.91% (p=0.000 n=38+34)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<3, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:14 ] 111ns ± 1%
106ns ± 1% -4.86% (p=0.020 n=39+2)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<10, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:23 ] 135ns ± 0%
130ns ± 0% -3.45% (p=0.020 n=35+2)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<31, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:46 ] 225ns ± 1%
223ns ± 0% -0.91% (p=0.003 n=37+2)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<100, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:120 ] 467ns ± 0%
472ns ± 0% +1.09% (p=0.003 n=38+2)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<1, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:12 ] 81.6ns ± 1%
74.8ns ± 0% -8.40% (p=0.000 n=37+33)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<3, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:14 ] 82.0ns ± 1%
74.8ns ± 0% -8.80% (p=0.000 n=37+32)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<10, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:21 ] 82.1ns ± 1%
74.9ns ± 0% -8.86% (p=0.000 n=35+34)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<31, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:42 ] 97.6ns ± 2%
91.8ns ± 0% -5.95% (p=0.000 n=35+27)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<100, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:111 ] 97.2ns ± 1%
91.2ns ± 2% -6.19% (p=0.000 n=37+38)
BM_HpackEncoderEncodeHeader<SingleNonInternedElem>/0/1
[framing_bytes/iter:54 header_bytes/iter:9 ] 230ns ± 0%
221ns ± 0% -3.91% (p=0.000 n=38+37)
BM_HpackEncoderEncodeHeader<MoreRepresentativeClientInitialMetadata>/0/16384
[framing_bytes/iter:9 header_bytes/iter:16 ] 206ns ± 2%
170ns ± 1% -17.51% (p=0.000 n=39+39)
BM_HpackEncoderEncodeHeader<RepresentativeServerInitialMetadata>/0/16384
[framing_bytes/iter:9 header_bytes/iter:3 ] 66.4ns ± 2%
62.5ns ± 1% -5.85% (p=0.000 n=34+39)
BM_HpackEncoderEncodeHeader<RepresentativeServerTrailingMetadata>/1/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 47.5ns ± 0%
45.9ns ± 1% -3.29% (p=0.000 n=26+38)
2019-09-06 13:43:29 -07:00
a9dbfb7c2f
Merge pull request #20028 from rmstar/ioscppe2e
...
Enable end2end C++ tests on iOS.
2019-09-06 13:27:13 -07:00
7ec1a9663d
Merge branch 'master' into move-method-handler
2019-09-06 12:21:29 -07:00
97f1f57dab
Allow call credentials to be set even after the call is created but before initial metadata is sent
2019-09-05 18:13:59 -07:00
9f02fc7f91
Enable end2end C++ tests on iOS.
...
Some e2e tests were disabled on iOS because they hit the Apple CFStream bug.
This commit enables e2e tests and works around the Apple bug by disabling CFStream.
2019-09-05 13:43:45 -07:00
9bb1247bdb
Merge pull request #16327 from vjpai/keep_running
...
C++ microbenchmarks: update from KeepRunning to range-based for
2019-09-05 11:07:53 -07:00
91274b0430
Merge pull request #20152 from AspirinSJL/group_test
...
Organize test cases for xds test
2019-09-04 13:41:33 -07:00
6751e6ea43
Merge pull request #20138 from rmstar/timejumptestfix
...
Don't run time_jump_test under sanitizers
2019-09-04 10:47:49 -07:00
c831a28dbd
Organize test cases for xds test
2019-09-03 16:47:25 -07:00
a1c0d3442f
Merge branch 'master' into move-method-handler
2019-09-03 14:47:45 -07:00
311bf04ec9
Merge pull request #19788 from chrisse74/master
...
remove all unused-parameter warnings in C++
2019-09-03 13:58:13 -07:00
6175294416
Merge pull request #20014 from arjunroy/on_hdr_nobranch
...
s/branch/tail_call/ for CH2 on_hdr().
2019-09-03 12:44:56 -07:00
c458f1cbbc
Merge pull request #20136 from grpc/revert-16452-add_msg_size_test
...
Revert "Test message size of 100MB"
2019-09-03 08:44:05 -07:00
ce29926334
Merge branch 'master' into move-method-handler
2019-09-03 08:08:18 -07:00
d9c9b52a68
Merge pull request #19507 from rmstar/ioscpptest_readme
...
Added documentation for C++ tests on iOS
2019-08-30 14:18:57 -07:00
fac072341c
Merge pull request #20126 from hcaseyal/bm_build
...
Remove build target for microbenchmark
2019-08-30 14:16:08 -07:00
b436758b14
Added documentation for C++ tests on iOS
2019-08-30 13:08:38 -07:00
2e1cb0d91a
Don't run time_jump_test under sanitizers
2019-08-30 11:43:21 -07:00
65eb9c9ddb
Revert "Test message size of 100MB"
2019-08-30 08:14:44 -07:00
d2b5fd75f0
Added test for time jumps
2019-08-29 22:41:06 -07:00
32801fb5eb
Remove build target for microbenchmark
2019-08-29 14:11:05 -07:00
1077b3435c
Use range-based for on state rather than state.KeepRunning when possible
2019-08-29 11:27:00 -07:00
be4318810c
Merge pull request #16452 from AspirinSJL/add_msg_size_test
...
Test message size of 100MB
2019-08-29 08:59:40 -07:00
fbd7ab53c7
Merge pull request #19036 from markdroth/lb_policy_per_rpc_backend_metric_api
...
Add API for accessing per-call backend metric data in LB policies.
2019-08-29 07:40:45 -07:00
d810671d93
Merge pull request #20039 from markdroth/health_check_service_name_change
...
Apply health check service name changes to existing subchannels.
2019-08-29 07:14:40 -07:00
d34f366337
Test message size of 100MB
2019-08-28 21:46:53 -07:00
4dfa808e75
Add test for timer expiry racing with cancelation
2019-08-28 17:09:42 -07:00
5f7f11e56e
Merge pull request #20022 from AspirinSJL/xds_test_refactor
...
Refactor response building in xds test
2019-08-28 16:22:29 -07:00
07ba4de392
Merge pull request #20091 from hcaseyal/bm_chttp2_transport
...
Fix race in bm_chttp2_transport
2019-08-28 16:13:23 -07:00
de2006d3e1
Merge branch 'master' into move-method-handler
2019-08-28 14:01:58 -07:00
b46e3668d3
s/branch/tail_call/ for CH2 on_hdr().
...
on_hdr() checks if a void-return function pointer is null before jumping to it.
If it is null, it returns an error; else it executes that function and returns
success.
This change converts the void-returning function to one that returns a
grpc_error* and thus saves a branch in on_hdr() (since we're branching once by
following the function pointer anyways, we're effectively coalescing these two
branches).
2019-08-28 13:06:11 -07:00
21c5424477
Add API for accessing per-call backend metric data in LB policies.
2019-08-28 12:48:06 -07:00
a1ca6a099d
Refactor response building in xds test
2019-08-28 12:44:31 -07:00
Jan Tattermusch