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

Drop usage of partial byte buf allocator. #5386

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Conversation

vietj
Copy link
Member

@vietj vietj commented Nov 5, 2024

Motivation:

Vert.x partial byte buf allocator implies the usage of unpooled buffers which can lead to increase significantly the memory footprint for SSL.

Changes:

Delete the partial byte buf allocator and use instead the same allocator than the non SSL case.

@vietj vietj added this to the 5.0.0 milestone Nov 5, 2024
@vietj vietj self-assigned this Nov 5, 2024
Motivation:

Vert.x partial byte buf allocator implies the usage of unpooled buffers which can lead to increase significantly the memory footprint for SSL.

Changes:

Delete the partial byte buf allocator and use instead the same allocator than the non SSL case.
@vietj vietj force-pushed the remove-partial-allocator branch from a14d29e to 559ac46 Compare November 6, 2024 07:42
@vietj vietj merged commit 55969ac into master Nov 7, 2024
8 checks passed
@vietj vietj deleted the remove-partial-allocator branch November 7, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants