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

[v23.2.x] Changed the partition_status chunked_fifo chunk size to 8 #16267

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #16247

@vbotbuildovich vbotbuildovich added this to the v23.2.x-next milestone Jan 24, 2024
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Jan 24, 2024
@piyushredpanda piyushredpanda modified the milestones: v23.2.x-next, v23.2.24 Jan 24, 2024
bharathv and others added 5 commits January 25, 2024 08:27
fetching partition information via health report API is heavy and
brokers API doesn't really use it. This patch avoids fetching that
information.

(cherry picked from commit 0ecb2a4)
Signed-off-by: Michal Maslanka <michal@redpanda.com>
(cherry picked from commit 9b5cc59)
Signed-off-by: Michal Maslanka <michal@redpanda.com>
(cherry picked from commit 22d34a6)
Signed-off-by: Michal Maslanka <michal@redpadna.com>
With large number of topics that contain a small amount of partitions we
may end up wasting a lot of memory using the default `ss::chunked_fifo`
chunk size of 128.
Change the chunk size of `partition_status` `chunked_fifo` collection to
8 to minimize a waste with large number of small topics.

Signed-off-by: Michal Maslanka <michal@redpanda.com>
(cherry picked from commit 75c9549)
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ bharathv
✅ mmaslankaprv
❌ Michal Maslanka


Michal Maslanka seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@mmaslankaprv mmaslankaprv merged commit 320f9cb into redpanda-data:v23.2.x Jan 25, 2024
22 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants