Assume validated

open-telemetry · Oct 6, 2021 · 97a91b9 · 97a91b9
1 parent 25a8c5e
commit 97a91b9
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 23 deletions.
diff --git a/...rc/main/java/io/opentelemetry/exporter/otlp/internal/grpc/DefaultGrpcExporterBuilder.java b/...rc/main/java/io/opentelemetry/exporter/otlp/internal/grpc/DefaultGrpcExporterBuilder.java
@@ -6,8 +6,6 @@
 package io.opentelemetry.exporter.otlp.internal.grpc;
 
 import static io.grpc.Metadata.ASCII_STRING_MARSHALLER;
-import static io.opentelemetry.api.internal.Utils.checkArgument;
-import static java.util.Objects.requireNonNull;
 
 import io.grpc.Codec;
 import io.grpc.ManagedChannel;
@@ -62,22 +60,17 @@ public DefaultGrpcExporterBuilder<T> setChannel(ManagedChannel channel) {
 
   @Override
   public DefaultGrpcExporterBuilder<T> setTimeout(long timeout, TimeUnit unit) {
-    requireNonNull(unit, "unit");
-    checkArgument(timeout >= 0, "timeout must be non-negative");
     timeoutNanos = unit.toNanos(timeout);
     return this;
   }
 
   @Override
   public DefaultGrpcExporterBuilder<T> setTimeout(Duration timeout) {
-    requireNonNull(timeout, "timeout");
     return setTimeout(timeout.toNanos(), TimeUnit.NANOSECONDS);
   }
 
   @Override
   public DefaultGrpcExporterBuilder<T> setEndpoint(String endpoint) {
-    requireNonNull(endpoint, "endpoint");
-
     URI uri;
     try {
       uri = new URI(endpoint);
@@ -97,10 +90,6 @@ public DefaultGrpcExporterBuilder<T> setEndpoint(String endpoint) {
 
   @Override
   public DefaultGrpcExporterBuilder<T> setCompression(String compressionMethod) {
-    requireNonNull(compressionMethod, "compressionMethod");
-    checkArgument(
-        compressionMethod.equals("gzip") || compressionMethod.equals("none"),
-        "Unsupported compression method. Supported compression methods include: gzip, none.");
     this.compressionEnabled = true;
     return this;
   }

diff --git a/...src/main/java/io/opentelemetry/exporter/otlp/internal/grpc/OkHttpGrpcExporterBuilder.java b/...src/main/java/io/opentelemetry/exporter/otlp/internal/grpc/OkHttpGrpcExporterBuilder.java
@@ -5,9 +5,6 @@
 
 package io.opentelemetry.exporter.otlp.internal.grpc;
 
-import static io.opentelemetry.api.internal.Utils.checkArgument;
-import static java.util.Objects.requireNonNull;
-
 import io.grpc.ManagedChannel;
 import io.opentelemetry.exporter.otlp.internal.Marshaler;
 import io.opentelemetry.exporter.otlp.internal.TlsUtil;
@@ -58,22 +55,17 @@ public OkHttpGrpcExporterBuilder<T> setChannel(ManagedChannel channel) {
 
   @Override
   public OkHttpGrpcExporterBuilder<T> setTimeout(long timeout, TimeUnit unit) {
-    requireNonNull(unit, "unit");
-    checkArgument(timeout >= 0, "timeout must be non-negative");
     timeoutNanos = unit.toNanos(timeout);
     return this;
   }
 
   @Override
   public OkHttpGrpcExporterBuilder<T> setTimeout(Duration timeout) {
-    requireNonNull(timeout, "timeout");
     return setTimeout(timeout.toNanos(), TimeUnit.NANOSECONDS);
   }
 
   @Override
   public OkHttpGrpcExporterBuilder<T> setEndpoint(String endpoint) {
-    requireNonNull(endpoint, "endpoint");
-
     URI uri;
     try {
       uri = new URI(endpoint);
@@ -93,10 +85,6 @@ public OkHttpGrpcExporterBuilder<T> setEndpoint(String endpoint) {
 
   @Override
   public OkHttpGrpcExporterBuilder<T> setCompression(String compressionMethod) {
-    requireNonNull(compressionMethod, "compressionMethod");
-    checkArgument(
-        compressionMethod.equals("gzip") || compressionMethod.equals("none"),
-        "Unsupported compression method. Supported compression methods include: gzip, none.");
     this.compressionEnabled = true;
     return this;
   }