Remove throws statements from the API until we decide how to document it (open-telemetry#2912)

bogdandrutu · web-flow · commit becded2181d4 · 2021-02-22T18:30:09.000-08:00
Signed-off-by: Bogdan Drutu &lt;bogdandrutu@gmail.com&gt;
diff --git a/api/all/src/main/java/io/opentelemetry/api/trace/SpanBuilder.java b/api/all/src/main/java/io/opentelemetry/api/trace/SpanBuilder.java
@@ -117,7 +117,6 @@ public interface SpanBuilder {
    *
    * @param context the {@code Context}.
    * @return this.
-   * @throws NullPointerException if {@code context} is {@code null}.
    */
   SpanBuilder setParent(Context context);
 
@@ -140,7 +139,6 @@ public interface SpanBuilder {
    *
    * @param spanContext the context of the linked {@code Span}.
    * @return this.
-   * @throws NullPointerException if {@code spanContext} is {@code null}.
    */
   SpanBuilder addLink(SpanContext spanContext);
 
@@ -154,8 +152,6 @@ public interface SpanBuilder {
    * @param spanContext the context of the linked {@code Span}.
    * @param attributes the attributes of the {@code Link}.
    * @return this.
-   * @throws NullPointerException if {@code spanContext} is {@code null}.
-   * @throws NullPointerException if {@code attributes} is {@code null}.
    */
   SpanBuilder addLink(SpanContext spanContext, Attributes attributes);
 
@@ -172,7 +168,6 @@ public interface SpanBuilder {
    * @param key the key for this attribute.
    * @param value the value for this attribute.
    * @return this.
-   * @throws NullPointerException if {@code key} is {@code null}.
    */
   SpanBuilder setAttribute(String key, String value);
 
@@ -186,7 +181,6 @@ public interface SpanBuilder {
    * @param key the key for this attribute.
    * @param value the value for this attribute.
    * @return this.
-   * @throws NullPointerException if {@code key} is {@code null}.
    */
   SpanBuilder setAttribute(String key, long value);
 
@@ -200,7 +194,6 @@ public interface SpanBuilder {
    * @param key the key for this attribute.
    * @param value the value for this attribute.
    * @return this.
-   * @throws NullPointerException if {@code key} is {@code null}.
    */
   SpanBuilder setAttribute(String key, double value);
 
@@ -214,7 +207,6 @@ public interface SpanBuilder {
    * @param key the key for this attribute.
    * @param value the value for this attribute.
    * @return this.
-   * @throws NullPointerException if {@code key} is {@code null}.
    */
   SpanBuilder setAttribute(String key, boolean value);
 
@@ -227,8 +219,6 @@ public interface SpanBuilder {
    * @param key the key for this attribute.
    * @param value the value for this attribute.
    * @return this.
-   * @throws NullPointerException if {@code key} is {@code null}.
-   * @throws NullPointerException if {@code value} is {@code null}.
    */
   <T> SpanBuilder setAttribute(AttributeKey<T> key, T value);
 
diff --git a/api/all/src/main/java/io/opentelemetry/api/trace/Tracer.java b/api/all/src/main/java/io/opentelemetry/api/trace/Tracer.java
@@ -65,7 +65,6 @@ public interface Tracer {
    *
    * @param spanName The name of the returned Span.
    * @return a {@code Span.Builder} to create and start a new {@code Span}.
-   * @throws NullPointerException if {@code spanName} is {@code null}.
    */
   SpanBuilder spanBuilder(String spanName);
 }

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,6 @@ public interface Tracer {`
`65`	`65`	`*`
`66`	`66`	`* @param spanName The name of the returned Span.`
`67`	`67`	`* @return a {@code Span.Builder} to create and start a new {@code Span}.`
`68`		`- * @throws NullPointerException if {@code spanName} is {@code null}.`
`69`	`68`	`*/`
`70`	`69`	`SpanBuilder spanBuilder(String spanName);`
`71`	`70`	`}`