Skip to content

chore(batch-import-worker): handle zero event batches#60473

Merged
z0br0wn merged 2 commits into
masterfrom
zbrown/import-worker-handle-zero-events
May 28, 2026
Merged

chore(batch-import-worker): handle zero event batches#60473
z0br0wn merged 2 commits into
masterfrom
zbrown/import-worker-handle-zero-events

Conversation

@z0br0wn
Copy link
Copy Markdown
Contributor

@z0br0wn z0br0wn commented May 28, 2026

Problem

When batch import worker processes chunks of zero events, it will forward an empty batch to capture and silently fail and pause the job.

Changes

  • When processing a batch of 0 events, short circuit on the capture call.
  • Also add some error propagation to the amplitude parsing, matching the other parser error paths.

@z0br0wn z0br0wn requested a review from a team May 28, 2026 16:28
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 28, 2026

Reviews (1): Last reviewed commit: "chore(batch-import-worker): handle zero ..." | Re-trigger Greptile

@z0br0wn z0br0wn merged commit 3b694f4 into master May 28, 2026
168 checks passed
@z0br0wn z0br0wn deleted the zbrown/import-worker-handle-zero-events branch May 28, 2026 16:58
@deployment-status-posthog
Copy link
Copy Markdown

deployment-status-posthog Bot commented May 28, 2026

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-05-28 17:28 UTC Run
prod-us ✅ Deployed 2026-05-28 17:52 UTC Run
prod-eu ✅ Deployed 2026-05-28 17:56 UTC Run

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.

2 participants