Revert demonstration bits

Author: jack-berg

api/all/src/main/java/io/opentelemetry/api/trace/Span.java

@@ -26,7 +26,7 @@
  * <p>{@code Span} <b>must</b> be ended by calling {@link #end()}.
  */
 @ThreadSafe
-public interface Span extends ImplicitContextKeyed, SpanParent {
+public interface Span extends ImplicitContextKeyed {
 
   /**
    * Returns the {@link Span} from the current {@link Context}, falling back to a default, no-op
@@ -339,6 +339,15 @@ default Span recordException(Throwable exception) {
     return recordException(exception, Attributes.empty());
   }
 
+  /**
+   * Records information about the {@link Throwable} to the {@link Span}.
+   *
+   * @param exception the {@link Throwable} to record.
+   * @param additionalAttributes the additional {@link Attributes} to record.
+   * @return this.
+   */
+  Span recordException(Throwable exception, Attributes additionalAttributes);
+
   /**
    * Updates the {@code Span} name.
    *

api/all/src/main/java/io/opentelemetry/api/trace/SpanParent.java

@@ -1,9 +1,2 @@
 Comparing source compatibility of opentelemetry-api-1.58.0-SNAPSHOT.jar against opentelemetry-api-1.57.0.jar
-***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.api.trace.Span  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	+++  NEW INTERFACE: io.opentelemetry.api.trace.SpanParent
-	---  REMOVED METHOD: PUBLIC(-) ABSTRACT(-) io.opentelemetry.api.trace.Span recordException(java.lang.Throwable, io.opentelemetry.api.common.Attributes)
-+++  NEW INTERFACE: PUBLIC(+) ABSTRACT(+) io.opentelemetry.api.trace.SpanParent  (not serializable)
-	+++  CLASS FILE FORMAT VERSION: 52.0 <- n.a.
-	+++  NEW SUPERCLASS: java.lang.Object
-	+++  NEW METHOD: PUBLIC(+) ABSTRACT(+) io.opentelemetry.api.trace.Span recordException(java.lang.Throwable, io.opentelemetry.api.common.Attributes)
+No changes.

docs/apidiffs/current_vs_latest/opentelemetry-api.txt

@@ -6,10 +6,6 @@ Comparing source compatibility of opentelemetry-sdk-trace-1.58.0-SNAPSHOT.jar ag
 ***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.sdk.trace.export.SimpleSpanProcessorBuilder  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.sdk.trace.export.SimpleSpanProcessorBuilder setMeterProvider(java.util.function.Supplier<io.opentelemetry.api.metrics.MeterProvider>)
-***  MODIFIED INTERFACE: PUBLIC ABSTRACT io.opentelemetry.sdk.trace.ReadWriteSpan  (not serializable)
-	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
-	===! UNCHANGED INTERFACE: io.opentelemetry.api.trace.Span
-	+++  NEW INTERFACE: io.opentelemetry.api.trace.SpanParent
 ***  MODIFIED CLASS: PUBLIC FINAL io.opentelemetry.sdk.trace.SdkTracerProviderBuilder  (not serializable)
 	===  CLASS FILE FORMAT VERSION: 52.0 <- 52.0
 	+++  NEW METHOD: PUBLIC(+) io.opentelemetry.sdk.trace.SdkTracerProviderBuilder setMeterProvider(java.util.function.Supplier<io.opentelemetry.api.metrics.MeterProvider>)

docs/apidiffs/current_vs_latest/opentelemetry-sdk-trace.txt

@@ -63,9 +63,9 @@ void verifyAllMethodsAreUnderTest() {
     assertThat(methods)
         .describedAs("all interface methods are being tested")
         .containsAll(allInterfaceMethods(Span.class));
-    //    assertThat(allInterfaceMethods(Span.class))
-    //        .describedAs("all tested methods are on the Span interface")
-    //        .containsAll(methods);
+    assertThat(allInterfaceMethods(Span.class))
+        .describedAs("all tested methods are on the Span interface")
+        .containsAll(methods);
   }
 
   @ParameterizedTest