Thanks to visit codestin.com
Credit goes to Github.com

Skip to content

fix: add ClickHouse client and InsertMany timeouts #700

Open
alexluong wants to merge 3 commits intomainfrom
fix-logmq
Open

fix: add ClickHouse client and InsertMany timeouts #700
alexluong wants to merge 3 commits intomainfrom
fix-logmq

Conversation

@alexluong
Copy link
Collaborator

No description provided.

alexluong and others added 2 commits February 12, 2026 03:18
Add message_id to the handler log entry, event_id, attempt_id, and
tenant_id after parsing in the batch processor. This makes it possible
to trace message redeliveries during pipeline stalls.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Add transport-level timeouts (DialTimeout: 10s, ReadTimeout: 30s) to the
ClickHouse client to prevent TCP-level hangs like the Feb 10 incident
where a hung write took 56 min to naturally fail via read: EOF.

Also add a 30s application-level context timeout to the InsertMany call
in the batch processor, bounding how long a single flush can block
regardless of transport behavior.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@vercel
Copy link

vercel bot commented Feb 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
outpost-docs Ready Ready Preview, Comment Feb 12, 2026 2:43pm
outpost-website Ready Ready Preview, Comment Feb 12, 2026 2:43pm

Request Review

Add insert duration timing to both success and error log lines
in processBatch, making it easier to spot slow ClickHouse inserts.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
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