Skip to content

sui_v1.20.0_1709244002_ci

@stefan-mysten stefan-mysten tagged this 29 Feb 20:50
## Description 

This PR enables logging of queries that time out. The logging uses the
`error` macro together with a new error code, `REQUEST_TIMEOUT`.

Note, that the timeout extension function was switched to `parse_query`
+ `execute`. Execute one takes care of the time, parse query takes care
of stringifying up the query (including resolving any passed, fragments,
etc).

## Test Plan 

Existing tests, manually checking for errors when setting a small
(500ms) timeout.

---
If your changes are not user-facing and do not break anything, you can
skip the following section. Otherwise, please briefly describe what has
changed under the Release Notes section.

### Type of Change (Check all that apply)

- [ ] protocol change
- [ ] user-visible impact
- [ ] breaking change for a client SDKs
- [ ] breaking change for FNs (FN binary must upgrade)
- [ ] breaking change for validators or node operators (must upgrade
binaries)
- [ ] breaking change for on-chain data layout
- [ ] necessitate either a data wipe or data migration

### Release notes
Enabled logging of queries that time out.
Assets 2
Loading