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

Decrease memory allocation rate #164

Closed
Jubilee101 opened this issue Dec 8, 2023 · 1 comment
Closed

Decrease memory allocation rate #164

Jubilee101 opened this issue Dec 8, 2023 · 1 comment
Assignees
Labels
enhancement Improvement to an existing feature

Comments

@Jubilee101
Copy link
Collaborator

Valgrind reports our base backup mallocs too many bytes. We need to go over base backup implementation and reuse some buffer, to decrease the number of bytes allocated.

@Jubilee101 Jubilee101 added the feature New feature label Dec 8, 2023
@Jubilee101 Jubilee101 self-assigned this Dec 8, 2023
@jesperpedersen jesperpedersen added enhancement Improvement to an existing feature and removed feature New feature labels Dec 8, 2023
@Jubilee101
Copy link
Collaborator Author

Heaptrack reports that most allocations come from untar, cjson parsing and extracting messages from the copy stream. We'll try to decrease the memory allocated for libarchive and cjson, and reuse one message struct for extraction. We will also try to use the stream buffer directly to hold the payload.

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

No branches or pull requests

2 participants