Proto commits in facebook/fbthrift

These 7 commits are when the Protocol Buffers files have changed:

2024-04-14

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.

2022-10-18

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

2021-05-10

Cover proto files with clang-format Reviewed By: h-friederich Differential Revision: D28314728 fbshipit-source-id: 244d828177cf6d5aa61469d2d6fea1eea898e2ec

2019-10-24

Yearless license header [4/n] Summary: This is all remaining headers that were autofixed by LICENSELINT. Differential Revision: D18085583 fbshipit-source-id: 93b08a2951021053a5e0a9d94f62d90a01703431

2019-05-17

(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

2018-12-12

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

2014-11-19

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