search

tikv / pd

Placement driver for TiKV
Apache License 2.0
1.04k stars 717 forks source link

Compare vtprotobuf + protobuf v2 with gogo protobuf #5414

Open rleungx opened 2 years ago

rleungx commented 2 years ago

Feature Request

Describe your feature request related problem

Currently, gogo protobuf is no longer maintained anymore. Maybe someday, we need to replace it with golang protobuf but there is a performance issue compared with gogo.

Describe the feature you'd like

From https://vitess.io/blog/2021-06-03-a-new-protobuf-generator-for-go, it has good performance and reduces a lot of CPU. Maybe we can use it in the future.

rleungx commented 2 years ago

BTW, we mixed use gogo and golang protobuf in our codebase right now. It's better to unify them at first.