These 10 commits are when the Protocol Buffers files have changed:
| Commit: | bf1bbff | |
|---|---|---|
| Author: | Ankit Kumar | |
| Committer: | meta-codesync[bot] | |
Remove unused StressTest.proto Summary: Deleted the unused StressTest.proto file and removed the corresponding StressTest_grpc gRPC library target from the BUCK file. Reviewed By: robertroeser Differential Revision: D88220455 fbshipit-source-id: 27cf9b9e31151a2a87746879c20d8c4a53be3d98
| Commit: | 3b4489b | |
|---|---|---|
| Author: | Ankit Kumar | |
| Committer: | meta-codesync[bot] | |
Microbenchmarks for comparing serialization and deserialization protocols Summary: Protocols to compare: Thrift Binary Thrift Compact Carbon Protobuf Reviewed By: robertroeser Differential Revision: D86432518 fbshipit-source-id: e45a0ab27ff7add79fae47b13a8425fd92401029
| Commit: | 48a16c0 | |
|---|---|---|
| Author: | Ankit Kumar | |
| Committer: | meta-codesync[bot] | |
StressTest: Add gRPC support for stress testing. Summary: Added async server and client for grpc to the stress test. Added support for two tests, Ping and Echo. Both of these would run on the completion queue worker (similar to iothread), for now. Reviewed By: robertroeser Differential Revision: D87117546 fbshipit-source-id: 2e142b43e71081518ebab80d6206be54fa2a4ea6
| Commit: | 6a48e4e | |
|---|---|---|
| Author: | Andres Suarez | |
| Committer: | Facebook GitHub Bot | |
Apply clang-format 18 Summary: Previously this code conformed from clang-format 12. Reviewed By: igorsugak Differential Revision: D56065247 fbshipit-source-id: f5a985dd8f8b84f2f9e1818b3719b43c5a1b05b3
The documentation is generated from this commit.
| Commit: | 1ff1ccd | |
|---|---|---|
| Author: | Chaoshuai Lu | |
| Committer: | Facebook GitHub Bot | |
Apply LicenseLint to fbcode/thrift Summary: Update the license to reflect the latest one we use. Reviewed By: iahs Differential Revision: D40322136 fbshipit-source-id: f27e1ed0cc1eca5cdbd2913b220cea0be0d51f57
| Commit: | de540c3 | |
|---|---|---|
| Author: | Andres Suarez | |
| Committer: | Facebook GitHub Bot | |
Cover proto files with clang-format Reviewed By: h-friederich Differential Revision: D28314728 fbshipit-source-id: 244d828177cf6d5aa61469d2d6fea1eea898e2ec
| Commit: | 6eb01ce | |
|---|---|---|
| Author: | Andres Suarez | |
| Committer: | Facebook Github Bot | |
Yearless license header [4/n] Summary: This is all remaining headers that were autofixed by LICENSELINT. Differential Revision: D18085583 fbshipit-source-id: 93b08a2951021053a5e0a9d94f62d90a01703431
| Commit: | 1da8d2e | |
|---|---|---|
| Author: | Songqiao Su | |
| Committer: | Facebook Github Bot | |
(Thrift) benchmarking Thrift and ProtoBuf on some more complex structure Summary: ProtoBuf version 3.6.1 and proto3 syntax was used in this experiment. Apart from the benchmark test already there in the repo, two kinds of new benchmarks are added. One is a large struct with many fields. The other is a nested map, since Protobuf doesn't support map to be the value a map (no map<key, map<key, value>>) and need to add a wrapper message in between, I did the same in Thrift to make it fair. Reviewed By: vitaut Differential Revision: D9381822 fbshipit-source-id: 9d9b08bc14aecbfe2b5e103a716697781fa04845
| Commit: | 19a87d8 | |
|---|---|---|
| Author: | Steve Gury | |
| Committer: | Facebook Github Bot | |
Fix protobuf compiler warning Summary: The protobuf compiler emits a warning because we don't specify the syntax used in the protobuf file. This diff simply fixes the warning. ``` No syntax specified for the proto file: thrift/lib/cpp2/test/ProtoBufBenchData.proto. Please use 'syntax = "proto2";' or 'syntax = "proto3";' to specify a syntax version. (Defaulted to proto2 syntax.) ``` Reviewed By: vitaut Differential Revision: D13420361 fbshipit-source-id: f3d9b5510f4b9a0bca070a73e8ec300c97fb456e
| Commit: | 50de816 | |
|---|---|---|
| Author: | Alex Landau | |
| Committer: | Dave Watson | |
Add ProtoBuf benchmarks Summary: The benchmarks are the same as used for Thrift, except without binary, since protobuf lacks this type. Test Plan: ``` ---- Thrift ------ -- ProtoBufs --- PB faster by Benchmark time/iter iters/s time/iter iters/s ========================================================================== write_Empty 123.64ns 8.09M 26.35ns 37.95M 369.10% read_Empty 58.92ns 16.97M 30.65ns 32.63M 92.28% write_SmallInt 146.19ns 6.84M 40.35ns 24.78M 262.28% read_SmallInt 78.38ns 12.76M 29.05ns 34.42M 169.75% write_BigInt 187.36ns 5.34M 50.04ns 19.98M 274.16% read_BigInt 125.69ns 7.96M 39.35ns 25.41M 219.22% write_SmallString 158.55ns 6.31M 57.82ns 17.30M 174.17% read_SmallString 109.20ns 9.16M 85.85ns 11.65M 27.18% write_BigString 399.54ns 2.50M 550.63ns 1.82M -27.20% read_BigString 394.01ns 2.54M 597.21ns 1.67M -34.25% write_BigBinary 314.72ns 3.18M read_BigBinary 223.87ns 4.47M write_LargeBinary 326.30ns 3.06M read_LargeBinary 242.07ns 4.13M write_Mixed 213.45ns 4.69M 67.70ns 14.77M 214.93% read_Mixed 175.83ns 5.69M 103.90ns 9.63M 69.24% write_SmallListInt 235.80ns 4.24M 77.20ns 12.95M 205.42% read_SmallListInt 252.46ns 3.96M 210.33ns 4.75M 19.95% write_BigListInt 100.26us 9.97K 21.02us 47.58K 377.23% read_BigListInt 121.96us 8.20K 37.66us 26.55K 223.78% write_BigListMixed 1.01ms 986.15 363.60us 2.75K 178.86% read_BigListMixed 1.54ms 649.58 2.37ms 422.59 -34.94% write_LargeListMixed 114.58ms 8.73 44.60ms 22.42 156.82% read_LargeListMixed 168.85ms 5.92 251.53ms 3.98 -32.77% ``` Reviewed By: davejwatson@fb.com Subscribers: alandau, bmatheny, mshneer FB internal diff: D1666012 Signature: t1:1666012:1415732917:c71938a4f24f522e67ec5beb9d8140500b7bc9b8