You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#### Description
The remote sampling functionality has been already removed from
`jaegerreceiver` in #14163. The logged warning has no impact or meaning.
#### Link to tracking issue
Fixes#35894
#### Testing
CI
---------
Signed-off-by: Yuri Shkuro <[email protected]>
Co-authored-by: Yang Song <[email protected]>
@@ -42,14 +40,6 @@ var disableJaegerReceiverRemoteSampling = featuregate.GlobalRegistry().MustRegis
42
40
featuregate.WithRegisterDescription("When enabled, the Jaeger Receiver will fail to start when it is configured with remote_sampling config. When disabled, the receiver will start and the remote_sampling config will be no-op."),
43
41
)
44
42
45
-
varonce sync.Once
46
-
47
-
funclogDeprecation(logger*zap.Logger) {
48
-
once.Do(func() {
49
-
logger.Warn("jaeger receiver will deprecate Thrift-gen and replace it with Proto-gen to be compatbible to jaeger 1.42.0 and higher. See https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/18485 for more details.")
50
-
})
51
-
}
52
-
53
43
// NewFactory creates a new Jaeger receiver factory.
54
44
funcNewFactory() receiver.Factory {
55
45
returnreceiver.NewFactory(
@@ -90,8 +80,6 @@ func createTracesReceiver(
90
80
cfg component.Config,
91
81
nextConsumer consumer.Traces,
92
82
) (receiver.Traces, error) {
93
-
logDeprecation(set.Logger)
94
-
95
83
// Convert settings in the source config to configuration struct
96
84
// that Jaeger receiver understands.
97
85
// Error handling for the conversion is done in the Validate function from the Config object itself.
0 commit comments