-
Notifications
You must be signed in to change notification settings - Fork 760
Closed
Labels
discussionIssue or PR that needs/is extended discussion.Issue or PR that needs/is extended discussion.priority:p2Issues that are less important than priority:p1Issues that are less important than priority:p1proto
Description
In #3070, we widened the supported version of protobuf to protobuf>=3.19, < 5.0 based on votes and discussion in #2880 (comment) and #3050
Long term, newer versions of protoc will not generate code that is compatible with older versions of protobuf library (see protocolbuffers/protobuf#11123 (comment)), meaning it will be hard to support both protobuf 3.x and 4.x. There are possible workarounds for this like
- generating code for both versions and choosing the correct one at runtime
- maintaining two major versions of
opentelemetry-protoand making releases against both major versions with separate protoc compilers.
The alternative is to drop support for protobuf 3 altogether at some reasonable point in the future, say 6 months.
Metadata
Metadata
Assignees
Labels
discussionIssue or PR that needs/is extended discussion.Issue or PR that needs/is extended discussion.priority:p2Issues that are less important than priority:p1Issues that are less important than priority:p1proto