Closed jsy1218 closed 1 month ago
This stack of pull requests is managed by Graphite. Learn more about stacking.
Join @jsy1218 and the rest of your teammates on Graphite
4 reviewers were added and 1 assignee was added to this PR based on 's automation.
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) feature
What is the current behavior? (You can also link to an open issue here) We don't have a way to shadow sample new Tenderly node endpoint that supports estimate gas bundle API
What is the new behavior (if this is a feature change)? We will have a quick implementation to shadow sample new Tenderly node endpoint that supports estimate gas bundle API.
Other information: We don't want the full-fledged implementation for both sampling and live traffic switching yet, because we don't know how much latency improvement Tenderly node endpoint brings over to routing-api. If it proves that the node endpoint is a lot faster than tenderly gateway endpoint, then we will implement the sampling and traffic switching feature inside routing-api, just like the view-only quoter one https://github.com/Uniswap/routing-api/blob/main/lib/handlers/quote/provider-migration/v3/traffic-switch-on-chain-quote-provider.ts. Hence the first requirement is - get the tenderly node endpoint latencies datapoints ASAP, with configurable shadow sampling percent, and which chain to enable.
Tested via local CLI https://app.warp.dev/block/k0YKya88coQWqOu6iWwZZZ, and can see success metrics:
and latencies improvement seems promising: