made txs shard handling stricter#50
Conversation
PR SummaryHigh Risk Overview The old per-endpoint Open-loop: when the scheduler owns the arrival clock, Plumbing: concurrency moves to Reviewed by Cursor Bugbot for commit 25d018e. Bugbot is set up for automated code reviews on this repo. Configure here. |
| attribute.String("endpoint", c.cfg.Endpoint), | ||
| attribute.String("scenario", tx.Scenario.Name), | ||
| )) | ||
| utils.SendOrDrop(c.receipts, tx) |
There was a problem hiding this comment.
Since this is SendOrDrop, under load we may drop some txs from receipt tracking. The tx is still submitted, but we'd never observe an on-chain failure for the dropped ones — so TrackReceipts is effectively a sample, not full coverage. Is that correct?
There was a problem hiding this comment.
it is. Also, afaiu in case of autobahn it will not be available at all since receipt store is not ready, right?
There was a problem hiding this comment.
In Autobahn the receipt store currently works for EVM tx (although there are many performance and safety issues the storage team is fixing), it doesn't work for Cosmos tx at all.
There was a problem hiding this comment.
anyway, after rebase receipt requests are gone entirely in favor of observing the finalized blocks.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit ead64e6. Configure here.

Txs in each shard will be sent sequentially, which is consistent with what autobahn expects.