Skip to content

Commit

Permalink
feedback
Browse files Browse the repository at this point in the history
Signed-off-by: Adrian Cole <adrian@tetrate.io>
  • Loading branch information
Adrian Cole committed Feb 14, 2024
1 parent e2015f2 commit 3fd2086
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,6 @@ static class BraveConfiguration {

@Bean
@ConditionalOnMissingBean(value = MutableSpan.class, parameterizedContainer = BytesEncoder.class)
@ConditionalOnEnabledTracing
BytesEncoder<MutableSpan> braveSpanEncoder(Encoding encoding,
ObjectProvider<Tag<Throwable>> throwableTagProvider) {
Tag<Throwable> throwableTag = throwableTagProvider.getIfAvailable(() -> Tags.ERROR);
Expand All @@ -173,7 +172,6 @@ static class OpenTelemetryConfiguration {

@Bean
@ConditionalOnMissingBean(value = Span.class, parameterizedContainer = BytesEncoder.class)
@ConditionalOnEnabledTracing
BytesEncoder<Span> zipkinSpanEncoder(Encoding encoding) {
return SpanBytesEncoder.forEncoding(encoding);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@
import zipkin2.reporter.Encoding;

import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
import org.springframework.boot.test.context.TestConfiguration;
import org.springframework.context.annotation.Bean;
import org.springframework.context.annotation.Configuration;

/**
* Configures the bean {@linkplain ZipkinAutoConfiguration} would from properties.
*/
@Configuration(proxyBeanMethods = false)
@TestConfiguration(proxyBeanMethods = false)
class DefaultEncodingConfiguration {

@Bean
Expand Down

0 comments on commit 3fd2086

Please sign in to comment.