Commit a708713
Carter Kozak
Simplify Tracer.shouldObserve to avoid JIT friction
Uses a simple condition rather than a switch statement
After:
`javap -c -p ./tracing/build/classes/java/main/com/palantir/tracing/Tracer.class`
```
private static boolean shouldObserve(com.palantir.tracing.Observability);
Code:
0: aload_0
1: getstatic #10 // Field com/palantir/tracing/Observability.SAMPLE:Lcom/palantir/tracing/Observability;
4: if_acmpeq 25
7: aload_0
8: getstatic #11 // Field com/palantir/tracing/Observability.UNDECIDED:Lcom/palantir/tracing/Observability;
11: if_acmpne 29
14: getstatic #12 // Field sampler:Lcom/palantir/tracing/TraceSampler;
17: invokeinterface #13, 1 // InterfaceMethod com/palantir/tracing/TraceSampler.sample:()Z
22: ifeq 29
25: iconst_1
26: goto 30
29: iconst_0
30: ireturn
```
Before:
`javap -c -p ./tracing/build/classes/java/main/com/palantir/tracing/Tracer.class`
```
private static boolean shouldObserve(com.palantir.tracing.Observability);
Code:
0: getstatic #10 // Field com/palantir/tracing/Tracer$1.$SwitchMap$com$palantir$tracing$Observability:[I
3: aload_0
4: invokevirtual #11 // Method com/palantir/tracing/Observability.ordinal:()I
7: iaload
8: tableswitch { // 1 to 3
1: 36
2: 38
3: 40
default: 49
}
36: iconst_1
37: ireturn
38: iconst_0
39: ireturn
40: getstatic #12 // Field sampler:Lcom/palantir/tracing/TraceSampler;
43: invokeinterface #13, 1 // InterfaceMethod com/palantir/tracing/TraceSampler.sample:()Z
48: ireturn
49: new #14 // class com/palantir/logsafe/exceptions/SafeIllegalArgumentException
52: dup
53: ldc #15 // String Unknown observability
55: iconst_1
56: anewarray #16 // class com/palantir/logsafe/Arg
59: dup
60: iconst_0
61: ldc #17 // String observability
63: aload_0
64: invokestatic #18 // Method com/palantir/logsafe/SafeArg.of:(Ljava/lang/String;Ljava/lang/Object;)Lcom/palantir/logsafe/SafeArg;
67: aastore
68: invokespecial #19 // Method com/palantir/logsafe/exceptions/SafeIllegalArgumentException."<init>":(Ljava/lang/String;[Lcom/palantir/logsafe/Arg;)V
71: athrow
```1 parent a998686 commit a708713
1 file changed
+2
-11
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | | - | |
30 | 29 | | |
31 | 30 | | |
32 | 31 | | |
| |||
86 | 85 | | |
87 | 86 | | |
88 | 87 | | |
89 | | - | |
90 | | - | |
91 | | - | |
92 | | - | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
| 88 | + | |
| 89 | + | |
99 | 90 | | |
100 | 91 | | |
101 | 92 | | |
| |||
0 commit comments