Skip to content

perf: small optimizations to reduce CPU overhead on hot paths - #190

Merged
mayurkale22 merged 4 commits into
mainfrom
small_improvements
Aug 22, 2025
Merged

perf: small optimizations to reduce CPU overhead on hot paths#190
mayurkale22 merged 4 commits into
mainfrom
small_improvements

Conversation

@mayurkale22

@mayurkale22 mayurkale22 commented Aug 9, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

This PR implements several targeted optimizations to reduce CPU overhead on hot paths. The changes focus on:

  • Minimizing data copying: Utilizing UnsafeByteOperations.unsafeWrap for request payload to avoid redundant ByteString copies.
  • Optimizing metrics: Precomputing OpenTelemetry Attributes to reduce allocation and overhead per metric record.

@mayurkale22
mayurkale22 requested a review from ath-08 August 9, 2025 02:10
ath-08
ath-08 previously approved these changes Aug 12, 2025
View reviewed changes
@mayurkale22
mayurkale22 merged commit f0d9777 into main Aug 22, 2025
16 checks passed
@release-please release-please Bot mentioned this pull request Aug 21, 2025
Merged
@ath-08
ath-08 deleted the small_improvements branch August 22, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pabloecol pabloecol pabloecol approved these changes
@ath-08 ath-08 ath-08 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mayurkale22 @pabloecol @ath-08