LiangliangSui commented on code in PR #2988:
URL: https://github.com/apache/fory/pull/2988#discussion_r2588015939
##########
benchmarks/cpp_benchmark/README.md:
##########
@@ -59,16 +59,21 @@ cmake --build . -j$(nproc)
## Benchmark Cases
-| Benchmark | Description
|
-| -------------------------------- |
------------------------------------------------------------------- |
-| `BM_Fory_Struct_Serialize` | Serialize a simple struct with 8 int32
fields using Fory |
-| `BM_Protobuf_Struct_Serialize` | Serialize the same struct using Protobuf
|
-| `BM_Fory_Struct_Deserialize` | Deserialize a simple struct using Fory
|
-| `BM_Protobuf_Struct_Deserialize` | Deserialize the same struct using
Protobuf |
-| `BM_Fory_Sample_Serialize` | Serialize a complex object with various
types and arrays using Fory |
-| `BM_Protobuf_Sample_Serialize` | Serialize the same object using Protobuf
|
-| `BM_Fory_Sample_Deserialize` | Deserialize a complex object using Fory
|
-| `BM_Protobuf_Sample_Deserialize` | Deserialize the same object using
Protobuf |
+| Benchmark | Description
|
+| ------------------------------------- |
------------------------------------------------------------------- |
+| `BM_Fory_Struct_Serialize` | Serialize a simple struct with 8
int32 fields using Fory |
+| `BM_Protobuf_Struct_Serialize` | Serialize the same struct using
Protobuf |
+| `BM_Fory_Struct_Deserialize` | Deserialize a simple struct using
Fory |
+| `BM_Protobuf_Struct_Deserialize` | Deserialize the same struct using
Protobuf |
+| `BM_Fory_Sample_Serialize` | Serialize a complex object with
various types and arrays using Fory |
+| `BM_Protobuf_Sample_Serialize` | Serialize the same object using
Protobuf |
+| `BM_Fory_Sample_Deserialize` | Deserialize a complex object using
Fory |
+| `BM_Protobuf_Sample_Deserialize` | Deserialize the same object using
Protobuf |
+| `BM_Fory_MediaContent_Serialize` | Serialize a complex object with
Media and Images using Fory |
+| `BM_Protobuf_MediaContent_Serialize` | Serialize the same object using
Protobuf |
+| `BM_Fory_MediaContent_Deserialize` | Deserialize a complex object with
Media and Images using Fory |
+| `BM_Protobuf_MediaContent_Deserialize`| Deserialize the same object using
Protobuf |
Review Comment:
Done. I think it's possible to configure Copilot to be included in the
reviewers by default, at least for document reviews, which would be helpful.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
