Use is_recording flag in jinja, celery, esearch, falcon instrumentati…

…ons (#1241)
open-telemetry · Oct 14, 2020 · b51e772 · b51e772
1 parent d602a1f
commit b51e772
Showing 1 changed file with 8 additions and 7 deletions.
diff --git a/...metry-instrumentation-pymemcache/src/opentelemetry/instrumentation/pymemcache/__init__.py b/...metry-instrumentation-pymemcache/src/opentelemetry/instrumentation/pymemcache/__init__.py
@@ -118,15 +118,16 @@ def _wrap_cmd(tracer, cmd, wrapped, instance, args, kwargs):
  _CMD, kind=SpanKind.INTERNAL, attributes={}
  ) as span:
  try:
- if not args:
- vals = ""
- else:
- vals = _get_query_string(args[0])
+ if span.is_recording():
+ if not args:
+ vals = ""
+ else:
+ vals = _get_query_string(args[0])
 
- query = "{}{}{}".format(cmd, " " if vals else "", vals)
- span.set_attribute(_RAWCMD, query)
+  query = "{}{}{}".format(cmd, " " if vals else "", vals)
+  span.set_attribute(_RAWCMD, query)
 
- _set_connection_attributes(span, instance)
+  _set_connection_attributes(span, instance)
  except Exception as ex: # pylint: disable=broad-except
  logger.warning(
  "Failed to set attributes for pymemcache span %s", str(ex)