Skip to content

Commit a5e6691

Browse files
committed
Update factories to accept TracerConfigurator
1 parent 2a97de9 commit a5e6691

File tree

4 files changed

+72
-6
lines changed

4 files changed

+72
-6
lines changed

sdk/include/opentelemetry/sdk/trace/tracer_context_factory.h

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,16 @@ class OPENTELEMETRY_EXPORT TracerContextFactory
5454
const opentelemetry::sdk::resource::Resource &resource,
5555
std::unique_ptr<Sampler> sampler,
5656
std::unique_ptr<IdGenerator> id_generator);
57+
58+
/**
59+
* Create a TracerContext.
60+
*/
61+
static std::unique_ptr<TracerContext> Create(
62+
std::vector<std::unique_ptr<SpanProcessor>> &&processors,
63+
const opentelemetry::sdk::resource::Resource &resource,
64+
std::unique_ptr<Sampler> sampler,
65+
std::unique_ptr<IdGenerator> id_generator,
66+
std::unique_ptr<TracerConfigurator> tracer_configurator);
5767
};
5868

5969
} // namespace trace

sdk/include/opentelemetry/sdk/trace/tracer_provider_factory.h

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ namespace trace
2828
class OPENTELEMETRY_EXPORT TracerProviderFactory
2929
{
3030
public:
31-
/* Serie of builders with a single processor. */
31+
/* Series of creator methods with a single processor. */
3232

3333
static std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> Create(
3434
std::unique_ptr<SpanProcessor> processor);
@@ -48,7 +48,14 @@ class OPENTELEMETRY_EXPORT TracerProviderFactory
4848
std::unique_ptr<Sampler> sampler,
4949
std::unique_ptr<IdGenerator> id_generator);
5050

51-
/* Serie of builders with a vector of processor. */
51+
static std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> Create(
52+
std::unique_ptr<SpanProcessor> processor,
53+
const opentelemetry::sdk::resource::Resource &resource,
54+
std::unique_ptr<Sampler> sampler,
55+
std::unique_ptr<IdGenerator> id_generator,
56+
std::unique_ptr<TracerConfigurator> tracer_configurator);
57+
58+
/* Series of creator methods with a single processor. */
5259

5360
static std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> Create(
5461
std::vector<std::unique_ptr<SpanProcessor>> &&processors);
@@ -68,6 +75,13 @@ class OPENTELEMETRY_EXPORT TracerProviderFactory
6875
std::unique_ptr<Sampler> sampler,
6976
std::unique_ptr<IdGenerator> id_generator);
7077

78+
static std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> Create(
79+
std::vector<std::unique_ptr<SpanProcessor>> &&processors,
80+
const opentelemetry::sdk::resource::Resource &resource,
81+
std::unique_ptr<Sampler> sampler,
82+
std::unique_ptr<IdGenerator> id_generator,
83+
std::unique_ptr<TracerConfigurator> tracer_configurator);
84+
7185
/* Create with a tracer context. */
7286

7387
static std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> Create(

sdk/src/trace/tracer_context_factory.cc

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,22 @@ std::unique_ptr<TracerContext> TracerContextFactory::Create(
5151
std::unique_ptr<Sampler> sampler,
5252
std::unique_ptr<IdGenerator> id_generator)
5353
{
54-
std::unique_ptr<TracerContext> context(new TracerContext(
55-
std::move(processors), resource, std::move(sampler), std::move(id_generator)));
54+
auto tracer_configurator =
55+
std::make_unique<TracerConfigurator>(TracerConfig::DefaultConfigurator());
56+
return Create(std::move(processors), resource, std::move(sampler), std::move(id_generator),
57+
std::move(tracer_configurator));
58+
}
59+
60+
std::unique_ptr<TracerContext> TracerContextFactory::Create(
61+
std::vector<std::unique_ptr<SpanProcessor>> &&processors,
62+
const opentelemetry::sdk::resource::Resource &resource,
63+
std::unique_ptr<Sampler> sampler,
64+
std::unique_ptr<IdGenerator> id_generator,
65+
std::unique_ptr<TracerConfigurator> tracer_configurator)
66+
{
67+
std::unique_ptr<TracerContext> context(
68+
new TracerContext(std::move(processors), resource, std::move(sampler),
69+
std::move(id_generator), std::move(tracer_configurator)));
5670
return context;
5771
}
5872

sdk/src/trace/tracer_provider_factory.cc

Lines changed: 30 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,10 +51,24 @@ std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> TracerProviderFactory
5151
const opentelemetry::sdk::resource::Resource &resource,
5252
std::unique_ptr<Sampler> sampler,
5353
std::unique_ptr<IdGenerator> id_generator)
54+
{
55+
auto tracer_configurator =
56+
std::make_unique<TracerConfigurator>(TracerConfig::DefaultConfigurator());
57+
return Create(std::move(processor), resource, std::move(sampler), std::move(id_generator),
58+
std::move(tracer_configurator));
59+
}
60+
61+
std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> TracerProviderFactory::Create(
62+
std::unique_ptr<SpanProcessor> processor,
63+
const opentelemetry::sdk::resource::Resource &resource,
64+
std::unique_ptr<Sampler> sampler,
65+
std::unique_ptr<IdGenerator> id_generator,
66+
std::unique_ptr<TracerConfigurator> tracer_configurator)
5467
{
5568
std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> provider(
5669
new opentelemetry::sdk::trace::TracerProvider(std::move(processor), resource,
57-
std::move(sampler), std::move(id_generator)));
70+
std::move(sampler), std::move(id_generator),
71+
std::move(tracer_configurator)));
5872
return provider;
5973
}
6074

@@ -87,10 +101,24 @@ std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> TracerProviderFactory
87101
const opentelemetry::sdk::resource::Resource &resource,
88102
std::unique_ptr<Sampler> sampler,
89103
std::unique_ptr<IdGenerator> id_generator)
104+
{
105+
auto tracer_configurator =
106+
std::make_unique<TracerConfigurator>(TracerConfig::DefaultConfigurator());
107+
return Create(std::move(processors), resource, std::move(sampler), std::move(id_generator),
108+
std::move(tracer_configurator));
109+
}
110+
111+
std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> TracerProviderFactory::Create(
112+
std::vector<std::unique_ptr<SpanProcessor>> &&processors,
113+
const opentelemetry::sdk::resource::Resource &resource,
114+
std::unique_ptr<Sampler> sampler,
115+
std::unique_ptr<IdGenerator> id_generator,
116+
std::unique_ptr<TracerConfigurator> tracer_configurator)
90117
{
91118
std::unique_ptr<opentelemetry::sdk::trace::TracerProvider> provider(
92119
new opentelemetry::sdk::trace::TracerProvider(std::move(processors), resource,
93-
std::move(sampler), std::move(id_generator)));
120+
std::move(sampler), std::move(id_generator),
121+
std::move(tracer_configurator)));
94122
return provider;
95123
}
96124

0 commit comments

Comments
 (0)