-
Notifications
You must be signed in to change notification settings - Fork 283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
don't notify scope listeners if trace is eligible for dropping #2661
Conversation
0f58f93
to
c461244
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Funny that you're using that byte for flags
. I happen to be doing the exact same thing for other purposes in my experiments...
dd-trace-core/src/main/java/datadog/trace/core/scopemanager/ContinuableScopeManager.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we have a quick discussion about the implications on other listeners than profiling?
c461244
to
c8eb1e7
Compare
Is there anything holding this back, or can it be merged? |
@bantonsson if you have changes in this area, please don't hold back to avoid a merge conflict. |
Ok. The only overlap is the |
No description provided.