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

[v24.1.x] Keep producer inflight requests queue bounded #23746

Merged

Conversation

vbotbuildovich
Copy link
Collaborator

Backport of PR #23692

Idempotent producers are allowed to produce up to 5 at a time, the five
requests outcome is cached in the producer for deduplication. Previously
the `_inflight_requests` list might grow unbounded if state machine
apply was lagging. This causes performance issues and unnecessary memory
pressure. Since the max inflight requests count is limited by producer
it is enough to keep only last 5 inflight requests in the
`_inflight_requests` list.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 755a890)
@vbotbuildovich vbotbuildovich added this to the v24.1.x-next milestone Oct 11, 2024
@vbotbuildovich vbotbuildovich added the kind/backport PRs targeting a stable branch label Oct 11, 2024
@vbotbuildovich
Copy link
Collaborator Author

vbotbuildovich commented Oct 11, 2024

the below tests from https://buildkite.com/redpanda/redpanda/builds/56333#01927aa3-fcbc-48a3-acff-052bbd99d73a have failed and will be retried

gtest_raft_rpunit
partition_balancer_simulator_test_rpunit

the below tests from https://buildkite.com/redpanda/redpanda/builds/57974#019321c7-4e8d-4957-a814-705475128b44 have failed and will be retried

gtest_raft_rpunit

@vbotbuildovich
Copy link
Collaborator Author

vbotbuildovich commented Oct 11, 2024

non flaky failures in https://buildkite.com/redpanda/redpanda/builds/56333#01927ae0-51b6-4b72-b6bf-2296ea352121:

"rptest.tests.idempotency_test.IdempotencyWriteCachingTest.test_idempotent_producers_write_caching"

non flaky failures in https://buildkite.com/redpanda/redpanda/builds/56333#01927ae1-317e-45e7-9bfe-c006e33f5696:

"rptest.tests.idempotency_test.IdempotencyWriteCachingTest.test_idempotent_producers_write_caching"

@vbotbuildovich
Copy link
Collaborator Author

vbotbuildovich commented Oct 11, 2024

Retry command for Build#56333

please wait until all jobs are finished before running the slash command

/ci-repeat 1
tests/rptest/tests/idempotency_test.py::IdempotencyWriteCachingTest.test_idempotent_producers_write_caching

Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit db884ac)
Signed-off-by: Michał Maślanka <michal@redpanda.com>
(cherry picked from commit 47d121a)
@mmaslankaprv mmaslankaprv force-pushed the backport-pr-23692-v24.1.x-481 branch from b7bae63 to e8dcf98 Compare November 12, 2024 19:07
@rockwotj rockwotj enabled auto-merge November 12, 2024 19:10
@vbotbuildovich
Copy link
Collaborator Author

@rockwotj rockwotj merged commit eab9fce into redpanda-data:v24.1.x Nov 12, 2024
17 checks passed
@BenPope BenPope modified the milestones: v24.1.x-next, v24.1.18 Nov 18, 2024
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.

4 participants