Skip to content

chore: Lower default relay batch timeout - #1417

Merged
dhfang merged 3 commits into
mainfrom
eric/default-relay-batch-timeout
Aug 26, 2026
Merged

chore: Lower default relay batch timeout#1417
dhfang merged 3 commits into
mainfrom
eric/default-relay-batch-timeout

Conversation

@Eric-Warehime

Copy link
Copy Markdown
Contributor

Closes https://linear.app/cosmoslabs/issue/FOU-1341/lower-default-batch-timeouts

3 is an arbitrary time I picked so lmk if there are suggestions or if we want to go even lower.

Signed-off-by: Eric Warehime <eric.warehime@gmail.com>
@Eric-Warehime
Eric-Warehime requested a review from a team as a code owner August 24, 2026 14:10
@linear-code

linear-code Bot commented Aug 24, 2026

Copy link
Copy Markdown

FOU-1341

@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Lowers the default receive and acknowledgement batch timeout for routes without explicit overrides.

  • Changes DefaultBatchTimeout from 10 seconds to 3 seconds.
  • Leaves the separate one-minute timeout-operation batch default unchanged.

Confidence Score: 5/5

The PR appears safe to merge because the shorter default only changes the intended batching latency and transaction-frequency tradeoff.

Routes without overrides will flush receive and acknowledgement batches sooner, while batch sizing, explicit overrides, timeout-operation batching, and relay correctness behavior remain unchanged.

Important Files Changed

Filename Overview
link/internal/relay/pipeline/opts.go Reduces the default receive and acknowledgement batch flush interval without changing override behavior or timeout-operation batching.

Reviews (1): Last reviewed commit: "Lower default relay batch timeout" | Re-trigger Greptile

@dhfang

dhfang commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

const DefaultPollInterval = 5 * time.Second

can we also reduce this to like 1s

@dhfang
dhfang merged commit e1e9bdb into main Aug 26, 2026
19 checks passed
@dhfang
dhfang deleted the eric/default-relay-batch-timeout branch August 26, 2026 03:33
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.

3 participants