-
-
Notifications
You must be signed in to change notification settings - Fork 205
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
respect IsSampled in SqlListener #1933
Conversation
src/Sentry.DiagnosticSource/Internals/DiagnosticSource/SentrySqlListener.cs
Outdated
Show resolved
Hide resolved
…qlListener.cs Co-authored-by: LucasZF <lucas-zimerman1@hotmail.com>
@lucas-zimerman yep. good point |
src/Sentry.DiagnosticSource/Internals/DiagnosticSource/SentrySqlListener.cs
Outdated
Show resolved
Hide resolved
src/Sentry.DiagnosticSource/Internals/DiagnosticSource/SentrySqlListener.cs
Outdated
Show resolved
Hide resolved
Not sure I understand the reason for this PR. Please add a better description to the top of this PR, or reference an issue if there is one open. Thanks. |
when looking into fixing #1350. i noticed that did i assume wrong? |
src/Sentry.DiagnosticSource/Internals/DiagnosticSource/SentrySqlListener.cs
Outdated
Show resolved
Hide resolved
After looking into this further, I see that the sampling decision is originally made in |
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.
Two minor code improvement suggestions, but looks good otherwise.
No description provided.