Added enableOpenTracing property

dimagi · Jan 12, 2022 · fcdceca · fcdceca
1 parent de8474a
commit fcdceca
Showing 1 changed file with 13 additions and 5 deletions.
diff --git a/src/main/java/org/javarosa/core/model/FormDef.java b/src/main/java/org/javarosa/core/model/FormDef.java
@@ -1109,8 +1109,10 @@ private void evaluateTriggerables(List<Triggerable> tv,
     private void evaluateTriggerable(Triggerable triggerable, TreeReference anchorRef) {
         // Contextualize the reference used by the triggerable against the anchor
         TreeReference contextRef = triggerable.narrowContextBy(anchorRef);
-        final Span span = GlobalTracer.get().activeSpan();
-        span.setTag("triggerable", triggerable.toString());
+        if (isTracingEnabled()) {
+            final Span span = GlobalTracer.get().activeSpan();
+            span.setTag("triggerable", triggerable.toString());
+        }
 
         // Now identify all of the fully qualified nodes which this triggerable
         // updates. (Multiple nodes can be updated by the same trigger)
@@ -1356,12 +1358,18 @@ public String fillTemplateString(String template, TreeReference contextRef, Hash
 
     @Trace
     public void populateDynamicChoices(ItemsetBinding itemset, TreeReference curQRef) {
-        final Span span = GlobalTracer.get().activeSpan();
-        span.setTag("itemset", itemset.nodesetRef.toString());
-        span.setTag("treeReference", curQRef.toString());
+        if (isTracingEnabled())) {
+            final Span span = GlobalTracer.get().activeSpan();
+            span.setTag("itemset", itemset.nodesetRef.toString());
+            span.setTag("treeReference", curQRef.toString());
+        }
         ItemSetUtils.populateDynamicChoices(itemset, curQRef, exprEvalContext, getMainInstance(), mProfilingEnabled);
     }
 
+    private boolean isTracingEnabled() {
+        return "true".equals(System.getProperty("src.main.java.org.javarosa.enableOpenTracing"));
+    }
+
     public String toString() {
         return getTitle();
     }