-
Notifications
You must be signed in to change notification settings - Fork 761
Description
Is your feature request related to a problem?
Protobuf version is still pinned to 3.x in open telemetry. This restricts Jina's protobuf version, even though we did efforts trying to support both versions of protobuf. Thus, the OpenTelemetry integration in jina enforces 3.x protobuf version and prevents users from benefiting from performance increase in 4.x
Describe the solution you'd like
We support merging the following PR: #2954
We believe that since OpenTelemetry is an opensource telemetry tool for cloud-native software, it should be a flexible dependency. Many cloud-native libraries rely one gRPC and would like to support latest protobuf
Describe alternatives you've considered
Just support 4.x and drop support for 3.x
Additional context
Basically in Jina, as an MLops framework, we wanted to support 4.x protobuf version without breaking compatibility with popular ml frameworks (paddle paddle, tensorflow,...).
Since, we wanted our network layer in the best shape possible, we made a PR to accept both versions and let the users decide which version they want.