You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For Propagators and other components returning invalid SpanContext instances it would be handy to share the SpanContext.getInvalid() instance, which is currently private.
If I remember correctly, there were a few reasons having keeping it private, mostly around users not abusing it. Do we want to change this?
(Observe this would be mostly an optimization. AnySpanContext can be created and users can check its getValid() method. On the other hand, TraceId and SpanId do expose their invalid instances).
For Propagators and other components returning invalid
SpanContext
instances it would be handy to share theSpanContext.getInvalid()
instance, which is currently private.If I remember correctly, there were a few reasons having keeping it private, mostly around users not abusing it. Do we want to change this?
(Observe this would be mostly an optimization. Any
SpanContext
can be created and users can check itsgetValid()
method. On the other hand,TraceId
andSpanId
do expose their invalid instances).See #767 for example.
The text was updated successfully, but these errors were encountered: