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.

Sign up for GitHub

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

Jump to bottom

[v24.3.x] cst: streaming parsing for tx range manifests #25084

Merged
merged 1 commit into from
Feb 14, 2025
Merged

[v24.3.x] cst: streaming parsing for tx range manifests #25084

merged 1 commit into from
Feb 14, 2025

Conversation

nvartolomei
Copy link
Contributor

@nvartolomei nvartolomei commented Feb 13, 2025 •
edited
Loading

Backport of PR #24728

Fixes #25082

@nvartolomei nvartolomei added this to the v24.3.x-next milestone Feb 13, 2025
@nvartolomei nvartolomei added the kind/backport PRs targeting a stable branch label Feb 13, 2025
@nvartolomei nvartolomei marked this pull request as ready for review February 13, 2025 18:46
@nvartolomei nvartolomei force-pushed the nv/manual-backport-24728-v24.3.x-49 branch from 00f47aa to dfc0490 Compare February 13, 2025 18:47
@nvartolomei
cst: streaming parsing for tx range manifests
dfc0490 
Gets rid of large allocs caused by rapidjson DOM parsing.

(cherry picked from commit 32905dc)
@nvartolomei nvartolomei requested a review from Lazin February 13, 2025 18:48
@vbotbuildovich
Copy link
Collaborator

CI test results

test results on build#61849
test_id test_kind job_url test_status passed
rptest.tests.archive_retention_test.CloudArchiveRetentionTest.test_delete.cloud_storage_type=CloudStorageType.ABS.retention_type=retention.bytes ducktape https://buildkite.com/redpanda/redpanda/builds/61849#019500e7-c2e4-401b-b125-18baa0618026 FLAKY 1/2

@nvartolomei
Copy link
Contributor Author

/dt
debug

@nvartolomei nvartolomei merged commit 1b91483 into redpanda-data:v24.3.x Feb 14, 2025
17 checks passed
@aanthony-rp aanthony-rp modified the milestones: v24.3.x-next, v24.3.6 Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Lazin Lazin Lazin approved these changes

Assignees
No one assigned
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Milestone
v24.3.6
Development

Successfully merging this pull request may close these issues.
4 participants
@nvartolomei @vbotbuildovich @Lazin @aanthony-rp