Remove ExtendedLogRecordBuilder#setException

Author: jack-berg

api/incubator/src/main/java/io/opentelemetry/api/incubator/logs/ExtendedDefaultLogger.java

@@ -40,11 +40,6 @@ public ExtendedLogRecordBuilder setEventName(String eventName) {
       return this;
     }
 
-    @Override
-    public ExtendedLogRecordBuilder setException(Throwable throwable) {
-      return this;
-    }
-
     @Override
     public LogRecordBuilder setTimestamp(long timestamp, TimeUnit unit) {
       return this;

api/incubator/src/main/java/io/opentelemetry/api/incubator/logs/ExtendedLogRecordBuilder.java

@@ -19,7 +19,4 @@ public interface ExtendedLogRecordBuilder extends LogRecordBuilder {
    * record with a non-empty event name is an Event.
    */
   ExtendedLogRecordBuilder setEventName(String eventName);
-
-  /** Set standard {@code exception.*} attributes based on the {@code throwable}. */
-  ExtendedLogRecordBuilder setException(Throwable throwable);
 }

sdk/logs/src/main/java/io/opentelemetry/sdk/logs/ExtendedSdkLogRecordBuilder.java

@@ -29,12 +29,6 @@ public ExtendedSdkLogRecordBuilder setEventName(String eventName) {
     return this;
   }
 
-  @Override
-  public ExtendedSdkLogRecordBuilder setException(Throwable throwable) {
-    super.setException(throwable);
-    return this;
-  }
-
   @Override
   public ExtendedSdkLogRecordBuilder setTimestamp(long timestamp, TimeUnit unit) {
     super.setTimestamp(timestamp, unit);

sdk/logs/src/main/java/io/opentelemetry/sdk/logs/SdkLogRecordBuilder.java

@@ -5,10 +5,6 @@
 
 package io.opentelemetry.sdk.logs;
 
-import static io.opentelemetry.sdk.internal.ExceptionAttributeResolver.EXCEPTION_MESSAGE;
-import static io.opentelemetry.sdk.internal.ExceptionAttributeResolver.EXCEPTION_STACKTRACE;
-import static io.opentelemetry.sdk.internal.ExceptionAttributeResolver.EXCEPTION_TYPE;
-
 import io.opentelemetry.api.common.AttributeKey;
 import io.opentelemetry.api.common.Value;
 import io.opentelemetry.api.logs.LogRecordBuilder;
@@ -17,7 +13,6 @@
 import io.opentelemetry.context.Context;
 import io.opentelemetry.sdk.common.InstrumentationScopeInfo;
 import io.opentelemetry.sdk.internal.AttributesMap;
-import io.opentelemetry.sdk.internal.ExceptionAttributeResolver;
 import java.time.Instant;
 import java.util.concurrent.TimeUnit;
 import javax.annotation.Nullable;
@@ -51,28 +46,6 @@ SdkLogRecordBuilder setEventName(String eventName) {
     return this;
   }
 
-  // accessible via ExtendedSdkLogRecordBuilder
-  SdkLogRecordBuilder setException(Throwable throwable) {
-    if (throwable == null) {
-      return this;
-    }
-    ExceptionAttributeResolver exceptionAttributeResolver =
-        loggerSharedState.getExceptionAttributeResolver();
-    String type = exceptionAttributeResolver.getExceptionType(throwable);
-    if (type != null) {
-      setAttribute(EXCEPTION_TYPE, type);
-    }
-    String message = exceptionAttributeResolver.getExceptionMessage(throwable);
-    if (message != null) {
-      setAttribute(EXCEPTION_MESSAGE, message);
-    }
-    String stacktrace = exceptionAttributeResolver.getExceptionStacktrace(throwable);
-    if (stacktrace != null) {
-      setAttribute(EXCEPTION_STACKTRACE, stacktrace);
-    }
-    return this;
-  }
-
   @Override
   public SdkLogRecordBuilder setTimestamp(long timestamp, TimeUnit unit) {
     this.timestampEpochNanos = unit.toNanos(timestamp);