open-telemetry
/
opentelemetry-java-instrumentation
mirror of https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixing 1.6 and perf issues

This commit is contained in:
Guillaume Polaert 2017-05-29 17:34:48 +02:00
parent 5e92d19114
commit 31c8f3279d
3 changed files with 13 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
dd-trace/pom.xml
View File

@ -13,6 +13,10 @@
<packaging>jar</packaging>
<name>dd-trace</name>
<properties>
<java.version>1.6</java.version>
</properties>
<dependencies>
<!-- Opentracing core -->

6
dd-trace/src/main/java/com/datadoghq/trace/DDSpanContext.java
View File

@ -175,8 +175,10 @@ public class DDSpanContext implements io.opentracing.SpanContext {
this.tags.put(tag, value);
//Call decorators
for (DDSpanContextDecorator decorator : tracer.getSpanContextDecorators(tag)) {
decorator.afterSetTag(this, tag, value);
if (tracer.getSpanContextDecorators(tag) != null) {
for (DDSpanContextDecorator decorator : tracer.getSpanContextDecorators(tag)) {
decorator.afterSetTag(this, tag, value);
}
}
//Error management
if (Tags.ERROR.getKey().equals(tag) && Boolean.TRUE.equals(value)) {

18
dd-trace/src/main/java/com/datadoghq/trace/DDTracer.java
View File

@ -38,7 +38,7 @@ public class DDTracer implements io.opentracing.Tracer {
/**
* Span context decorators
*/
private final Map<String, List<DDSpanContextDecorator>> spanContextDecorators = new HashMap<>();
private final Map<String, List<DDSpanContextDecorator>> spanContextDecorators = new HashMap<String, List<DDSpanContextDecorator>>();
private final static Logger logger = LoggerFactory.getLogger(DDTracer.class);
@ -78,12 +78,8 @@ public class DDTracer implements io.opentracing.Tracer {
* @return the list of span context decorators
*/
public List<DDSpanContextDecorator> getSpanContextDecorators(String tag) {
List<DDSpanContextDecorator> decorators = Collections.emptyList();
String key = getHashKey(tag);
if (spanContextDecorators.containsKey(key)) {
decorators = Collections.unmodifiableList(spanContextDecorators.get(key));
}
List<DDSpanContextDecorator> decorators = spanContextDecorators.get(tag);
return decorators;
}
@ -94,15 +90,14 @@ public class DDTracer implements io.opentracing.Tracer {
* @param decorator The decorator in the list
*/
public void addDecorator(DDSpanContextDecorator decorator) {
String key = getHashKey(decorator.getMatchingTag());
List<DDSpanContextDecorator> list = spanContextDecorators.get(key);
List<DDSpanContextDecorator> list = spanContextDecorators.get(decorator.getMatchingTag());
if (list == null) {
list = new ArrayList<>();
list = new ArrayList<DDSpanContextDecorator>();
}
list.add(decorator);
spanContextDecorators.put(key, list);
spanContextDecorators.put(decorator.getMatchingTag(), list);
}
@ -327,9 +322,6 @@ public class DDTracer implements io.opentracing.Tracer {
}
private String getHashKey(String tag) {
return tag;
}
private static class CodecRegistry {