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

Enabling NOOP classes to work with pre-existing production code #263

Merged
merged 1 commit into from
Mar 29, 2018
Merged

Enabling NOOP classes to work with pre-existing production code #263

merged 1 commit into from
Mar 29, 2018

Commits on Mar 29, 2018

  1. Enabling NOOP classes to work with pre-existing production code 

    In special circumstances, code may be written to automatically create spans,
negating the need to always check if Tracer.activeSpan() is null. Under these
circumstances, the Noop Tracer and ScopeManager will cause NPEs when they return
null values when their `active` values are accessed from the GlobalTracer. This
prevents code from running successfully when using Noop Tracers.

This minor change allows users to depend that their code will genuinely Noop
under all use cases.
    @natehart
    natehart committed Mar 29, 2018
    Configuration menu
    Copy the full SHA
    cc63126 View commit details
    Browse the repository at this point in the history