Skip to content
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.

Sign up for GitHub

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

Jump to bottom

"UseSampling" is hard to discover. Consider moving it into the same namespace as TelemetryProcessorsChainBuilder #84

Closed
SergeyKanzhelev opened this issue Oct 21, 2015 · 2 comments
Closed

"UseSampling" is hard to discover. Consider moving it into the same namespace as TelemetryProcessorsChainBuilder #84

SergeyKanzhelev opened this issue Oct 21, 2015 · 2 comments
Assignees
@abaranch
Labels
bug
Milestone
95

Comments

@SergeyKanzhelev
Copy link
Contributor

Consider moving UseSampling into the same namespace as telemetryProcessorsChainBuilder to simplify discovery of this method
@SergeyKanzhelev
Copy link
Contributor Author

@upendras, @vitalyf007 what do you think of moving it to the namespace Microsoft.ApplicationInsights.Extensibility?

@upendras
Copy link
Member

makes sense...right now you have to look at documentation to find it...

@AlexBulankou AlexBulankou added this to the 93 milestone Dec 3, 2015
@AlexBulankou AlexBulankou added bug p2 and removed bug labels Dec 3, 2015
@abaranch abaranch mentioned this issue Dec 7, 2015
Merged
@abaranch abaranch closed this as completed Dec 7, 2015
TimothyMothra pushed a commit that referenced this issue Oct 25, 2019
Merge pull request #84 from karolz-ms/dev/karolz/tpl-events
8fdcf12 
Suppress TPL events for EventSource and ETW modules
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abaranch abaranch

Labels
bug
Projects
None yet
Milestone
95
Development

No branches or pull requests
4 participants
@upendras @SergeyKanzhelev @abaranch @AlexBulankou