Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,7 @@ public static OpenTelemetry get() {
* OpenTelemetrySdk.builder().buildAndRegisterGlobal()} instead of calling this method directly.
*/
public static void set(OpenTelemetry openTelemetry) {
Objects.requireNonNull(openTelemetry, "openTelemetry");
synchronized (mutex) {
if (globalOpenTelemetry != null) {
throw new IllegalStateException(
Expand All @@ -192,6 +193,7 @@ public static void set(OpenTelemetry openTelemetry) {
* @since 1.52.0
*/
public static void set(Supplier<OpenTelemetry> supplier) {
Objects.requireNonNull(supplier, "supplier");
synchronized (mutex) {
OpenTelemetry openTelemetry = supplier.get();
set(openTelemetry);
Expand Down Expand Up @@ -386,6 +388,7 @@ public ContextPropagators getPropagators() {

@Override
public TracerBuilder tracerBuilder(String instrumentationScopeName) {
Objects.requireNonNull(instrumentationScopeName, "instrumentationScopeName");
return delegate.tracerBuilder(instrumentationScopeName);
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
package io.opentelemetry.api.baggage;

import com.google.auto.value.AutoValue;
import io.opentelemetry.api.internal.ApiUsageLogger;
import io.opentelemetry.common.ApiUsageLogger;
import javax.annotation.concurrent.Immutable;

@Immutable
Expand All @@ -25,7 +25,7 @@ abstract class ImmutableEntryMetadata implements BaggageEntryMetadata {
*/
static ImmutableEntryMetadata create(String metadata) {
if (metadata == null) {
ApiUsageLogger.log("metadata is null");
ApiUsageLogger.logNullParam(BaggageEntryMetadata.class, "create", "metadata");
return EMPTY;
}
return new AutoValue_ImmutableEntryMetadata(metadata);
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@

import io.opentelemetry.api.common.AttributeKey;
import io.opentelemetry.api.common.Value;
import io.opentelemetry.common.ApiUsageLogger;
import io.opentelemetry.context.Context;
import java.time.Instant;
import java.util.concurrent.TimeUnit;
Expand Down Expand Up @@ -59,26 +60,46 @@ public LogRecordBuilder setObservedTimestamp(Instant instant) {

@Override
public LogRecordBuilder setContext(Context context) {
if (context == null) {
ApiUsageLogger.logNullParam(LogRecordBuilder.class, "setContext", "context");
return this;
}
return this;
}

@Override
public LogRecordBuilder setSeverity(Severity severity) {
if (severity == null) {
ApiUsageLogger.logNullParam(LogRecordBuilder.class, "setSeverity", "severity");
return this;
}
return this;
}

@Override
public LogRecordBuilder setSeverityText(String severityText) {
if (severityText == null) {
ApiUsageLogger.logNullParam(LogRecordBuilder.class, "setSeverityText", "severityText");
return this;
}
return this;
}

@Override
public LogRecordBuilder setBody(String body) {
if (body == null) {
ApiUsageLogger.logNullParam(LogRecordBuilder.class, "setBody", "body");
return this;
}
return this;
}

@Override
public LogRecordBuilder setBody(Value<?> body) {
if (body == null) {
ApiUsageLogger.logNullParam(LogRecordBuilder.class, "setBody", "body");
return this;
}
return this;
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,8 @@

package io.opentelemetry.api.logs;

import java.util.Objects;

class DefaultLoggerProvider implements LoggerProvider {

private static final LoggerProvider INSTANCE = new DefaultLoggerProvider();
Expand All @@ -18,18 +20,21 @@ static LoggerProvider getInstance() {

@Override
public LoggerBuilder loggerBuilder(String instrumentationScopeName) {
Objects.requireNonNull(instrumentationScopeName, "instrumentationScopeName");
return NOOP_BUILDER;
}

private static class NoopLoggerBuilder implements LoggerBuilder {

@Override
public LoggerBuilder setSchemaUrl(String schemaUrl) {
Objects.requireNonNull(schemaUrl, "schemaUrl");
return this;
}

@Override
public LoggerBuilder setInstrumentationVersion(String instrumentationVersion) {
Objects.requireNonNull(instrumentationVersion, "instrumentationVersion");
return this;
}

Expand Down
Loading
Loading