Closed as not planned
Closed as not planned
Description
Use case(s) - what problem will this feature solve?
opentelemetry-go1 supports protobuf over gRPC and over HTTP, but its proto file defines both the service and related messages in the same file.
When using protobuf over HTTP, it's necessary to remove the gRPC dependency to reduce the binary size.
Proposed Solution
Add an option to rename grpc file.GoImportPath 5
- g := gen.NewGeneratedFile(filename, file.GoImportPath)
+ path := file.GoImportPath
+ if n, ok := rename[file.Desc.Path()]; ok {
+ path = n
+ }
+ g := gen.NewGeneratedFile(filename, path)
-grpc_rename log.proto=github.com/aaa/bbb/xxx -grpc_rename trace.proto=github.com/aaa/bbb/yyy
Alternatives Considered
- opentelemetry-proto split proto file
- opentelemetry-proto-go add same shim file Feat split grpc again open-telemetry/opentelemetry-proto-go#248