-
Notifications
You must be signed in to change notification settings - Fork 572
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
Fix issue #154: Add ability to change sampling rate #190
Conversation
…we need to make custom profiler
Codecov Report
@@ Coverage Diff @@
## main #190 +/- ##
==========================================
- Coverage 57.26% 57.12% -0.13%
==========================================
Files 73 73
Lines 2936 2957 +21
==========================================
+ Hits 1681 1689 +8
- Misses 1112 1122 +10
- Partials 143 146 +3
Continue to review full report at Codecov.
|
var DefaultProfileTypes = []ProfileType{ProfileCPU, ProfileAllocObjects, ProfileAllocSpace, ProfileInuseObjects, ProfileInuseSpace} | ||
|
||
type Config struct { | ||
ApplicationName string // e.g backend.purchases | ||
ServerAddress string // e.g http://pyroscope.services.internal:4040 | ||
AuthToken string // specify this token when using pyroscope cloud | ||
SampleRate uint32 |
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.
SampleRate uint32 | |
// SampleRate uint32 |
I think we should comment this out until we implement golang profiler support (#191)
…r defining the common const or variables; 3. make the spyname to a const variable
make the sample rate configured for python, ruby and go, but it doesn't take affect for go, because we use the the 'runtime profile' of golang, we need to find the way to build the customer profiler for go.