Skip to content

Commit 2442da8

Browse files
authored
Merge branch 'main' into support-custom-baggage-header-prefix
2 parents 2afb58a + 315abae commit 2442da8

File tree

4 files changed

+29
-6
lines changed

4 files changed

+29
-6
lines changed

package.json

+1-3
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22
"name": "opentelemetry",
33
"version": "0.1.0",
44
"description": "OpenTelemetry is a distributed tracing and stats collection framework.",
5-
"main": "build/src/index.js",
6-
"types": "build/src/index.d.ts",
75
"scripts": {
86
"precompile": "lerna run version",
97
"compile": "tsc --build",
@@ -65,7 +63,7 @@
6563
"lerna-changelog": "1.0.1",
6664
"markdownlint-cli": "0.29.0",
6765
"semver": "7.3.5",
68-
"typedoc": "0.22.9",
66+
"typedoc": "0.22.10",
6967
"typescript": "4.4.4",
7068
"update-ts-references": "2.4.1"
7169
},

packages/opentelemetry-sdk-trace-base/src/Span.ts

+4
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,10 @@ export class Span implements api.Span, ReadableSpan {
131131
startTime?: api.TimeInput
132132
): this {
133133
if (this._isSpanEnded()) return this;
134+
if (this._spanLimits.eventCountLimit === 0) {
135+
api.diag.warn('No events allowed.');
136+
return this;
137+
}
134138
if (this.events.length >= this._spanLimits.eventCountLimit!) {
135139
api.diag.warn('Dropping extra events.');
136140
this.events.shift();

packages/opentelemetry-sdk-trace-base/test/common/Span.test.ts

+22
Original file line numberDiff line numberDiff line change
@@ -621,6 +621,28 @@ describe('Span', () => {
621621
assert.strictEqual(span.events[span.events.length - 1].name, 'sent149');
622622
});
623623

624+
it('should add no event', () => {
625+
const tracer = new BasicTracerProvider({
626+
spanLimits: {
627+
eventCountLimit: 0,
628+
},
629+
}).getTracer('default');
630+
631+
const span = new Span(
632+
tracer,
633+
ROOT_CONTEXT,
634+
name,
635+
spanContext,
636+
SpanKind.CLIENT
637+
);
638+
for (let i = 0; i < 10; i++) {
639+
span.addEvent('sent' + i);
640+
}
641+
span.end();
642+
643+
assert.strictEqual(span.events.length, 0);
644+
});
645+
624646
it('should set an error status', () => {
625647
const span = new Span(
626648
tracer,

tsconfig.json

+2-3
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
"extends": "./tsconfig.base.json",
33
"files": [],
44
"typedocOptions": {
5-
"packages": [
6-
"experimental/packages/opentelemetry-api-metrics",
5+
"entryPointStrategy": "packages",
6+
"entryPoints": [
77
"experimental/packages/opentelemetry-exporter-metrics-otlp-grpc",
88
"experimental/packages/opentelemetry-exporter-metrics-otlp-http",
99
"experimental/packages/opentelemetry-exporter-metrics-otlp-proto",
@@ -26,7 +26,6 @@
2626
"packages/opentelemetry-propagator-b3",
2727
"packages/opentelemetry-propagator-jaeger",
2828
"packages/opentelemetry-resources",
29-
"packages/opentelemetry-sdk-metrics-base",
3029
"packages/opentelemetry-sdk-trace-base",
3130
"packages/opentelemetry-sdk-trace-node",
3231
"packages/opentelemetry-sdk-trace-web",

0 commit comments

Comments
 (0)