Skip to content

Conversation

@comphead
Copy link
Contributor

@comphead comphead commented Apr 3, 2025

Which issue does this PR close?

Related to #7179
Closes #.

Rationale for this change

Adding a profiling profile to investigate cpu/mem usage for benches. The profile is release inherited but with debug symbols enabled

Example using Samply tool

# run first time to compile 
cargo bench --bench parse_decimal --profile profiling
# second time with profiler to avoid compilation noise
samply record cargo bench --bench parse_decimal --profile profiling

What changes are included in this PR?

Are there any user-facing changes?

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @comphead -- I have also found samply to be super helpful

@alamb alamb added the development-process Related to development process of arrow-rs label Apr 3, 2025
@alamb alamb merged commit 6c71a80 into apache:main Apr 3, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development-process Related to development process of arrow-rs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants