Skip to content
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

symbolizer: reduce the number of debuginfo copies #5330

Merged

Conversation

danipozo
Copy link
Contributor

Check the existence of debuginfo files in the temporary directory to reduce the number of copies on each request, which drives high disk usage and latency.

@danipozo danipozo requested a review from a team as a code owner November 27, 2024 16:18
Copy link
Member

@brancz brancz left a comment

Choose a reason for hiding this comment

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

this needs a go fmt as well, otherwise the strategy lgtm

pkg/symbolizer/symbolizer.go Outdated Show resolved Hide resolved
@danipozo danipozo force-pushed the reduce-debuginfo-copies-when-symbolizing branch from 4762250 to a2198b8 Compare December 9, 2024 10:48
@danipozo danipozo force-pushed the reduce-debuginfo-copies-when-symbolizing branch from a2198b8 to 4c37617 Compare December 12, 2024 10:12
@brancz
Copy link
Member

brancz commented Jan 14, 2025

Not sure why CI isn't running on this PR. Can you rebase maybe and see if that re-triggers CI?

@danipozo
Copy link
Contributor Author

Rebased but it's not triggering the CI

@brancz
Copy link
Member

brancz commented Jan 14, 2025

Looks like it's running now!

@brancz
Copy link
Member

brancz commented Jan 15, 2025

One small thing the linter is complaining about, then I think this is ready to merge.

@danipozo
Copy link
Contributor Author

Seems like Go / Test on arm64 timed out

@brancz brancz merged commit 84cc112 into parca-dev:main Jan 16, 2025
33 of 34 checks passed
@brancz
Copy link
Member

brancz commented Jan 16, 2025

Merged, that job is not very reliable and super slow and I don't want to slow this down further since I don't think this is any different for arm64 and x86.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants