Yash Tibrewal
|
f44e0c07a7
|
Reviewer comments
|
5 years ago |
Soheil Hassas Yeganeh
|
39e982a263
|
Merge pull request #19488 from soheilhy/string-view2
Introduce string_view and use it for gpr_split_host_port
|
5 years ago |
|
Yash Tibrewal
|
420d5413c7
|
Use the actual formula
|
5 years ago |
Moiz Haidry
|
1c354e7c1f
|
Move grpc async, callback and sync implementation to grpc_impl namespace
|
5 years ago |
rmstar
|
8ce8cfe4c3
|
Merge pull request #19439 from rmstar/enable_cfstream
Enable CFStream by default on iOS
|
5 years ago |
|
Yash Tibrewal
|
464f558a45
|
Increase the control message size
|
5 years ago |
mgravell
|
0a1147b58c
|
found another way to nop the native-call
|
5 years ago |
Jan Tattermusch
|
6854789cdb
|
Merge pull request #19504 from jtattermusch/upport_v1_22_x
Upport changes from 1.22.x to master
|
5 years ago |
|
mgravell
|
e4411e03e6
|
added by mistake
|
5 years ago |
|
mgravell
|
f53d844da9
|
attempt to fix the utf-8 encode benchmark; not currently working
|
5 years ago |
|
mgravell
|
dbef6c9c70
|
add utf-8 encode benchmark
|
5 years ago |
|
mgravell
|
f5091b2622
|
add UTF8-decode benchmark
| Method | Job | Runtime | PayloadSize | Mean | Error | StdDev | Gen 0 | Gen 1 | Gen 2 | Allocated |
|------- |----- |-------- |------------ |-------------:|-----------:|-----------:|-------:|------:|------:|----------:|
| Run | Clr | Clr | 0 | 1.736 ns | 0.0101 ns | 0.0094 ns | - | - | - | - |
| Run | Core | Core | 0 | 1.306 ns | 0.0108 ns | 0.0095 ns | - | - | - | - |
| Run | Clr | Clr | 1 | 35.384 ns | 0.2282 ns | 0.2135 ns | 0.0101 | - | - | 64 B |
| Run | Core | Core | 1 | 32.388 ns | 0.3333 ns | 0.2955 ns | 0.0101 | - | - | 64 B |
| Run | Clr | Clr | 4 | 57.736 ns | 0.3889 ns | 0.3448 ns | 0.0114 | - | - | 72 B |
| Run | Core | Core | 4 | 52.878 ns | 0.2802 ns | 0.2621 ns | 0.0114 | - | - | 72 B |
| Run | Clr | Clr | 128 | 554.819 ns | 4.4341 ns | 4.1477 ns | 0.0830 | - | - | 530 B |
| Run | Core | Core | 128 | 336.356 ns | 1.6148 ns | 1.4315 ns | 0.0835 | - | - | 528 B |
| Run | Clr | Clr | 1024 | 4,050.850 ns | 28.9245 ns | 25.6408 ns | 0.6016 | - | - | 3820 B |
| Run | Core | Core | 1024 | 2,272.534 ns | 33.8963 ns | 31.7066 ns | 0.6016 | - | - | 3808 B |
|
5 years ago |
|
mgravell
|
52de8a0a17
|
ShortRunJob *added* a test!
|
5 years ago |
|
mgravell
|
05a0dd20e4
|
convert micro-benchmarks to benchmarkdotnet
|
5 years ago |
|
Marc Gravell
|
cb813e1ffc
|
check the *actual* length to allow more stack usage *and* allow smaller pool rentals
|
6 years ago |
|
Marc Gravell
|
74be06c80f
|
remove UTF8 byte[] allocations: decode directly into a string; encode using stack or array-pool
|
6 years ago |
Arjun Roy
|
b6514a6438
|
Merge pull request #19487 from arjunroy/static_md_idx_v2
Pre-compute static metadata index for hpack_encoder.
|
6 years ago |
|
Arjun Roy
|
d527c1fbda
|
Pre-compute static metadata index for hpack_encoder.
Originally, hpack_encoder would check if a metadata was static or not by
comparing its pointer to the known static metadata global table and checking if
it was within bounds. This check was performed regardless of if the metadata was
static or not, and is somewhat costly.
Instead, we now pre-compute the static metadata index during code generation
time, and store it with static metadata objects. We read that value only if we
are dealing with a static metadata flag (which we know from the storage type of
the grpc_mdelem).
This yields slightly faster metadata encoding:
BM_HpackEncoderEncodeHeader<EmptyBatch>/0/16384
[framing_bytes/iter:9 header_bytes/iter:0 ] 34.9ns ± 2%
34.2ns ± 1% -2.04% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<EmptyBatch>/1/16384
[framing_bytes/iter:9 header_bytes/iter:0 ] 34.9ns ± 2%
34.2ns ± 1% -2.01% (p=0.000 n=20+19)
BM_HpackEncoderEncodeHeader<SingleStaticElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.6ns ± 0%
49.2ns ± 2% -2.74% (p=0.000 n=18+20)
BM_HpackEncoderEncodeHeader<SingleInternedKeyElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:6 ] 84.7ns ± 1%
83.5ns ± 1% -1.43% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<SingleInternedElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.4ns ± 0%
47.9ns ± 0% -4.83% (p=0.000 n=18+17)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<1, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 51.1ns ± 2%
48.9ns ± 1% -4.32% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<3, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.8ns ± 2%
48.8ns ± 2% -3.88% (p=0.000 n=19+20)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<10, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.2ns ± 1%
47.9ns ± 0% -4.47% (p=0.000 n=19+16)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<31, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.2ns ± 0%
47.9ns ± 0% -4.46% (p=0.000 n=18+16)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<100, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.2ns ± 0%
47.9ns ± 0% -4.40% (p=0.000 n=19+17)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<1, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.7ns ± 2%
48.8ns ± 2% -3.81% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<3, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.9ns ± 2%
48.8ns ± 2% -4.05% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<10, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.1ns ± 0%
48.0ns ± 1% -4.27% (p=0.000 n=17+17)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<31, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.1ns ± 0%
48.0ns ± 1% -4.28% (p=0.000 n=18+17)
BM_HpackEncoderEncodeHeader<SingleInternedBinaryElem<100, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.1ns ± 0%
48.0ns ± 0% -4.33% (p=0.000 n=18+17)
BM_HpackEncoderEncodeHeader<SingleNonInternedElem>/0/16384
[framing_bytes/iter:9 header_bytes/iter:9 ] 91.4ns ± 1%
90.7ns ± 1% -0.79% (p=0.000 n=18+20)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<1, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:12 ] 116ns ± 1%
116ns ± 1% -0.46% (p=0.002 n=20+20)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<3, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:14 ] 122ns ± 0%
121ns ± 0% -0.69% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<10, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:23 ] 144ns ± 1%
144ns ± 0% -0.23% (p=0.009 n=20+20)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<31, false>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:46 ] 232ns ± 0%
232ns ± 1% -0.26% (p=0.021 n=18+19)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<1, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:12 ] 92.9ns ± 1%
92.0ns ± 1% -0.97% (p=0.000 n=19+19)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<3, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:14 ] 94.0ns ± 1%
92.6ns ± 1% -1.45% (p=0.000 n=20+19)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<10, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:21 ] 93.9ns ± 2%
92.8ns ± 1% -1.17% (p=0.001 n=20+19)
BM_HpackEncoderEncodeHeader<SingleNonInternedBinaryElem<100, true>>/0/16384
[framing_bytes/iter:9 header_bytes/iter:111 ] 106ns ± 0%
105ns ± 3% -1.15% (p=0.000 n=18+20)
BM_HpackEncoderEncodeHeader<SingleNonInternedElem>/0/1
[framing_bytes/iter:81 header_bytes/iter:9 ] 355ns ± 1%
354ns ± 0% -0.35% (p=0.015 n=19+20)
BM_HpackEncoderEncodeHeader<RepresentativeClientInitialMetadata>/0/16384
[framing_bytes/iter:9 header_bytes/iter:8.00002 ] 139ns ± 1%
133ns ± 1% -4.46% (p=0.000 n=19+20)
BM_HpackEncoderEncodeHeader<MoreRepresentativeClientInitialMetadata>/0/16384
[framing_bytes/iter:9 header_bytes/iter:16 ] 236ns ± 1%
231ns ± 1% -2.24% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<RepresentativeServerInitialMetadata>/0/16384
[framing_bytes/iter:9 header_bytes/iter:3 ] 73.6ns ± 1%
70.5ns ± 1% -4.14% (p=0.000 n=20+20)
BM_HpackEncoderEncodeHeader<RepresentativeServerTrailingMetadata>/1/16384
[framing_bytes/iter:9 header_bytes/iter:1 ] 50.5ns ± 0%
49.2ns ± 2% -2.60% (p=0.000 n=16+20)
|
6 years ago |
|
Arjun Roy
|
920bacf11d
|
Merge pull request #19498 from arjunroy/hpack_parser_on_hdr_v2
Codegen optimizations for hpack_parser on_hdr().
|
6 years ago |
Guantao Liu
|
f83d6ced54
|
Merge branch 'master' into no_backup_poller
|
6 years ago |
Juanli Shen
|
4c96998eb1
|
Fix typo
|
6 years ago |
|
rmstar
|
ce282e7d7e
|
Merge pull request #19395 from rmstar/c++_on_ios
Fix C++ tests to run on iOS
|
6 years ago |
Yang Gao
|
a0e76c0300
|
Merge pull request #19484 from yang-g/oauth
Update oauth2 token endpoints
|
6 years ago |
|
Jan Tattermusch
|
8fe1abebbf
|
Merge pull request #19419 from mit-mit/bumpdart
Roll Dart to 2.3
|
6 years ago |
|
Jan Tattermusch
|
0a43bd0008
|
Merge pull request #19503 from iamrare/patch-1
fix link in doc
|
6 years ago |
Jan Tattermusch
|
3222a350e5
|
Merge remote-tracking branch 'upstream/v1.22.x' into HEAD
|
6 years ago |
モハメド
|
743be2ba94
|
fix link
|
6 years ago |
Guantao Liu
|
9f59029aee
|
Revert the implementation of grpc_iomgr_run_in_background() for gevent,
which causes duplicate definitions.
|
6 years ago |
apolcyn
|
58ed35407a
|
Merge pull request #19489 from apolcyn/fix_ares_throttle
Fix DNS resolver cooldown
|
6 years ago |
|
Soheil Hassas Yeganeh
|
ea63c00d38
|
Revert "Fix build failure in credential_test.cc"
This reverts commit dc858eea25.
|
6 years ago |
|
Soheil Hassas Yeganeh
|
fedf7e373e
|
Fix a backward compatibility bug.
To remain backward compatible, we only set port if it's a non-emptry
string. But, we always store host.
|
6 years ago |
|
Soheil Hassas Yeganeh
|
c9ec1a64ed
|
Fix SplitHostPort for empty strings.
Add unittests to catch such errors in the future.
|
6 years ago |
|
Soheil Hassas Yeganeh
|
dbf88dd66f
|
Revert "Revert "Introduce string_view and use it for gpr_split_host_port.""
This reverts commit 80c177d4c4.
|
6 years ago |
nanahpang
|
3a33fccb3a
|
Merge pull request #19499 from nanahpang/fix_typo_cl_255487586
fix typo "transfered" to "transferred"
|
6 years ago |
|
Arjun Roy
|
80d1aec021
|
Codegen optimizations for hpack_parser on_hdr.
Currently, the log statements for on_hdr clobber some registers even if it is
disabled by default. Additionally, the no-error path is doing some unnecessary
jumps. This PR separates out and un-inlines the logging code, and marks the
no-error case as likely to occur.
This results in slightly faster hpack_parsing, e.g.:
BM_HpackParserParseHeader<IndexedSingleStaticElem, UnrefHeader>
23.8ns ± 0% 19.8ns ± 0% -16.81% (p=0.000 n=17+17)
BM_HpackParserParseHeader<AddIndexedSingleStaticElem, UnrefHeader>
160ns ± 0% 154ns ± 0% -3.18% (p=0.000 n=19+20)
BM_HpackParserParseHeader<KeyIndexedSingleStaticElem, UnrefHeader>
216ns ± 1% 214ns ± 2% -0.70% (p=0.004 n=18+18)
BM_HpackParserParseHeader<IndexedSingleInternedElem, UnrefHeader>
35.4ns ± 0% 34.4ns ± 0% -2.93% (p=0.000 n=16+16)
BM_HpackParserParseHeader<RepresentativeClientInitialMetadata, UnrefHeader>
140ns ± 0% 130ns ± 0% -7.06% (p=0.000 n=17+18)
BM_HpackParserParseHeader<MoreRepresentativeClientInitialMetadata, UnrefHeader>
644ns ± 1% 636ns ± 2% -1.29% (p=0.003 n=17+20)
BM_HpackParserParseHeader<RepresentativeServerInitialMetadata, UnrefHeader>
49.1ns ± 0% 38.7ns ± 0% -21.13% (p=0.000 n=19+18)
BM_HpackParserParseHeader<RepresentativeServerTrailingMetadata, UnrefHeader>
47.1ns ± 0% 43.2ns ± 0% -8.17% (p=0.000 n=20+19)
BM_HpackParserParseHeader<RepresentativeClientInitialMetadata, OnInitialHeader>
452ns ± 1% 417ns ± 1% -7.86% (p=0.000 n=20+20)
BM_HpackParserParseHeader<MoreRepresentativeClientInitialMetadata,
OnInitialHeader>
1.06µs ± 1% 1.02µs ± 2% -3.42% (p=0.000 n=19+20)
BM_HpackParserParseHeader<RepresentativeServerInitialMetadata, OnInitialHeader>
156ns ± 0% 142ns ± 1% -9.08% (p=0.000 n=17+19)
BM_HpackParserParseHeader<SameDeadline, OnHeaderTimeout>
117ns ± 0% 113ns ± 1% -3.98% (p=0.000 n=20+20)
|
6 years ago |
yunjiaw26
|
c83c28167e
|
Merge pull request #19393 from yunjiaw26/extend_thd_stack_size
Extend thread class to accept setting stack size as a option
|
6 years ago |
|
nanahpang
|
36339f4032
|
fix typo "transfered" to "transferred"
find the typo during import and create this pr to sync up
|
6 years ago |
|
apolcyn
|
dd5f0fcf48
|
Merge pull request #19482 from qixuanl1/add-pick-first-unary
add pick_first_unary interop test
|
6 years ago |
Muxi Yan
|
a3a9480c49
|
Merge pull request #19492 from grpc/oauth2-uninitialized
Resolve uninitialized warning in oauth2_credentials
|
6 years ago |
|
Alexander Polcyn
|
0d2c622f9e
|
Fix DNS resolver cooldown
|
6 years ago |
|
Soheil Hassas Yeganeh
|
3f574aecc9
|
Merge pull request #19493 from soheilhy/fix-cred-test
Fix build failure in credential_test.cc
|
6 years ago |
Yunjia Wang
|
266c919afc
|
Merge remote-tracking branch 'upstream/master' into extend_thd_stack_size
|
6 years ago |
|
Soheil Hassas Yeganeh
|
dc858eea25
|
Fix build failure in credential_test.cc
|
6 years ago |
|
Muxi Yan
|
424328b8e7
|
Resolve uninitialized warning
|
6 years ago |
Karthik Ravi Shankar
|
0d2d88fce5
|
Merge pull request #19446 from karthikravis/continue-migration-to-impl
Move more of usage to grpc_impl
|
6 years ago |
|
Yunjia Wang
|
b95bb89d13
|
Add TODO comment
|
6 years ago |
Vijay Pai
|
858b9117dd
|
Merge pull request #19478 from grpc/vjpai-patch-1
Explain order of callback execution in executor comments
|
6 years ago |
|
Guantao Liu
|
7610817bc8
|
Also add the missing implementation of grpc_iomgr_run_in_background()
for gevent.
|
6 years ago |
|
Arjun Roy
|
15e78eb8b5
|
Merge pull request #19485 from arjunroy/hpack_encoder_ensure_space
Slightly better codegen for hpack_encoder.
|
6 years ago |
jboeuf
|
bd8698e8fd
|
Merge pull request #19032 from jboeuf/sts_core_creds_pr
Adding support for STS Token Exchange Creds in core:
|
6 years ago |
