Skip to content

feat(taskworker): Send uncompressed profile tasks to worker #95783

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
merged 3 commits into from
Jul 17, 2025

Conversation

enochtangg
Copy link
Member

The process_profiles worker code now supports uncompressed parameter payloads (#95692). This PR is responsible for removing double compression by changing the process_profile call site to send uncompressed tasks to taskworker as taskworker already handles zstd compression in its platform.

@enochtangg enochtangg requested a review from a team as a code owner July 17, 2025 14:40
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 17, 2025
@enochtangg enochtangg requested a review from a team July 17, 2025 14:48
Copy link

codecov bot commented Jul 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #95783    +/-   ##
========================================
  Coverage   80.88%   80.89%            
========================================
  Files       10578    10582     +4     
  Lines      610329   610444   +115     
  Branches    23950    23950            
========================================
+ Hits       493684   493803   +119     
+ Misses     115937   115933     -4     
  Partials      708      708            

@enochtangg enochtangg merged commit 5151349 into master Jul 17, 2025
64 checks passed
@enochtangg enochtangg deleted the remove-zlib-compression-profiles branch July 17, 2025 19:25
cursor bot pushed a commit that referenced this pull request Jul 18, 2025
The process_profiles worker code now supports uncompressed parameter
payloads (#95692). This PR is
responsible for removing double compression by changing the
process_profile call site to send uncompressed tasks to taskworker as
taskworker already handles zstd compression in its platform.
andrewshie-sentry pushed a commit that referenced this pull request Jul 21, 2025
The process_profiles worker code now supports uncompressed parameter
payloads (#95692). This PR is
responsible for removing double compression by changing the
process_profile call site to send uncompressed tasks to taskworker as
taskworker already handles zstd compression in its platform.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants