search

weaveworks / common

Libraries used in multiple Weave projects
Other
129 stars 92 forks source link

Proposal: remove dependency on deprecated gogo/protobuf #268

Open krajorama opened 1 year ago

krajorama commented 1 year ago

Remove dependency on https://github.com/gogo/protobuf .

This would be a breaking change, thus requires a major version bump.

bboreham commented 1 year ago

The reason for picking gogo/protobuf was performance: I would like to be confident that we are moving to one with equal or better performance.

(It shouldn't be a massive ask for httpgrpc which is essentially transporting one string, but still)