Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Commit

Permalink
Use a String for the reference type
Browse files Browse the repository at this point in the history
  • Loading branch information
Ben Sigelman committed Jul 15, 2016
1 parent ff2d9ff commit 0e5a342
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion opentracing-api/src/main/java/io/opentracing/Tracer.java
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ interface SpanBuilder {
*
* @see io.opentracing.References
*/
SpanBuilder addReference(Object referenceType, SpanContext referencedContext);
SpanBuilder addReference(String referenceType, SpanContext referencedContext);

/** Same as {@link Span#setTag(String, String)}, but for the span being built. */
SpanBuilder withTag(String key, String value);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ abstract class AbstractSpanBuilder implements Tracer.SpanBuilder {
protected abstract AbstractSpan createSpan();

@Override
public final Tracer.SpanBuilder addReference(Object referenceType, SpanContext referredTo) {
public final Tracer.SpanBuilder addReference(String referenceType, SpanContext referredTo) {
this.references.add(new Reference(referenceType, referredTo));
return this;
}
Expand Down Expand Up @@ -88,10 +88,10 @@ public final Span start() {
}

public static final class Reference {
private Object referenceType;
private String referenceType;
private SpanContext referredTo;

Reference(Object type, SpanContext referredTo) {
Reference(String type, SpanContext referredTo) {
this.referenceType = type;
this.referredTo = referredTo;
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ final class NoopSpanBuilder implements Tracer.SpanBuilder {
private NoopSpanBuilder() {}

@Override
public Tracer.SpanBuilder addReference(Object refType, SpanContext referenced) {
public Tracer.SpanBuilder addReference(String refType, SpanContext referenced) {
return this;
}

Expand Down

0 comments on commit 0e5a342

Please sign in to comment.