-
-
Notifications
You must be signed in to change notification settings - Fork 464
Profiling - Deduplication and cleanup #4681
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
Merged
lbloder
merged 32 commits into
feat/continuous-profiling-02
from
feat/continuous-profiling-03
Sep 26, 2025
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
bb12990
add readme and info about commit of the source repository
lbloder 14d2f3b
delete jfr file on jvm exit
lbloder bdd7b3a
further split into smaller methods
lbloder 1b88a3d
deduplicate frames in order to save bandwidth, add converter tests
lbloder 81aef38
remove Platform Enum, use string constants instead for compatibility …
lbloder 8f32429
implement equals and hashcode for SentryStackFrame to make frame dedu…
lbloder b7645f2
bump api
lbloder 66ae530
improve error handling, fix start stop start flow
lbloder 037b237
add new testfile
lbloder 2979b16
calculate ticksPerNanosecond in constructor
lbloder a32b71e
adapt Ratelimiter to check for both ProfileChunk and ProfileChunkUi r…
lbloder 4b1e57e
update ratelimiter test to check for both profileChunk and profileChu…
lbloder 8992018
use string constant instead of string
lbloder c9a6bc0
Format code
getsentry-bot 451f191
add non aggregating event collector to send each event individually, …
lbloder 885f0ab
adapt converter tests to new non-aggregated converter
lbloder 7ae393e
Merge branch 'feat/continuous-profiling-03' of github.com:getsentry/s…
lbloder b1701c5
Format code
getsentry-bot 30cb14b
add logging to loadProfileConverter
lbloder 94a37b1
Format code
getsentry-bot 524a32a
fix duplication of events
lbloder 2cfe307
catch all exception happening when converting from jfr
lbloder a559bb3
add exists and writable info to log message
lbloder 94a0097
add method to safely delete file
lbloder 0179ea3
remove setNative call
lbloder 66a6c33
fix test
lbloder bb10259
fix reference to commit we vendored from
lbloder 7a9c931
drop event if it cannot be processed to not lose the whole chunk
lbloder bcb7feb
make format
lbloder 0ea8a2d
fix test
lbloder 2e54dde
Format code
getsentry-bot 242ca41
Profiling - OTEL profiling fix, Stabilization, Logging (#4746)
lbloder File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next
Next commit
add readme and info about commit of the source repository
- Loading branch information
commit bb12990f8c16381344c9b50e4f87ab0234925489
There are no files selected for viewing
4 changes: 4 additions & 0 deletions
4
...c-profiler/src/main/java/io/sentry/asyncprofiler/vendor/asyncprofiler/README.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| # Vendored AsyncProfiler code for converting JFR Files | ||
| - Vendored-in from commit fe1bc66d4b6181413847f6bbe5c0db805f3e9194 of repository: [email protected]:async-profiler/async-profiler.git | ||
| - Only the code related to JFR conversion is included. | ||
| - The `AsyncProfiler` itself is included as a dependency in the Maven project. | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.