Skip to content

feat(code-mappings): Add batch splitting for large uploads#3210

Draft
romtsn wants to merge 2 commits intorz/feat/code-mappings-api-integrationfrom
rz/feat/code-mappings-batching
Draft

feat(code-mappings): Add batch splitting for large uploads#3210
romtsn wants to merge 2 commits intorz/feat/code-mappings-api-integrationfrom
rz/feat/code-mappings-batching

Conversation

@romtsn
Copy link
Member

@romtsn romtsn commented Mar 12, 2026

#skip-changelog

Split large mapping files into batches of 300 (the backend limit) per request.
Each batch is sent sequentially with progress reporting, and results are merged
into a single summary.

Also changes the output table to only show error rows — for large uploads
(hundreds of mappings), printing every row would flood the terminal. Successful
mappings are reflected in the summary counts instead.

Stack: #3207#3208#3209#3210

Backend PRs: getsentry/sentry#109783, getsentry/sentry#109785, getsentry/sentry#109786

@github-actions
Copy link
Contributor

github-actions bot commented Mar 12, 2026

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 8735962

@romtsn romtsn force-pushed the rz/feat/code-mappings-batching branch from 7cb21c6 to 5b0bce3 Compare March 12, 2026 10:55
@romtsn romtsn force-pushed the rz/feat/code-mappings-api-integration branch from f5ab182 to 37fbb57 Compare March 12, 2026 10:55
@romtsn romtsn force-pushed the rz/feat/code-mappings-batching branch from 5b0bce3 to a7afb81 Compare March 12, 2026 11:11
@romtsn romtsn force-pushed the rz/feat/code-mappings-api-integration branch 2 times, most recently from 5720ec2 to bbc4d47 Compare March 12, 2026 11:30
@romtsn romtsn force-pushed the rz/feat/code-mappings-batching branch from a7afb81 to 3fcf380 Compare March 12, 2026 11:30
romtsn and others added 2 commits March 12, 2026 12:43
Split large mapping files into batches of 300 (the backend limit)
per request. Each batch is sent sequentially with progress reporting.
Results are merged into a single summary table.

Batch-level HTTP failures are captured without aborting remaining
batches, and the final exit code reflects any errors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
For large uploads (hundreds of mappings), printing every row floods
the terminal. Show only failed mappings in the table since those are
actionable; successful ones are already reflected in the summary counts.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@romtsn romtsn force-pushed the rz/feat/code-mappings-batching branch from 3fcf380 to 8735962 Compare March 12, 2026 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant