Record timing information for failed GraphQL queries #3508

lutter · 2022-04-27T20:51:02Z

No description provided.

server/http/src/service.rs

server/http/src/request.rs

This didn't need to be a future; just a synchronous helper function is enough

Before, we weren't recording failed queries at all

lutter force-pushed the lutter/query-err branch 2 times, most recently from 7edb798 to c7c4c76 Compare April 27, 2022 21:21

evaporei reviewed Apr 28, 2022

View reviewed changes

server/http/src/service.rs Show resolved Hide resolved

evaporei reviewed Apr 28, 2022

View reviewed changes

server/http/src/request.rs Show resolved Hide resolved

evaporei approved these changes Apr 28, 2022

View reviewed changes

lutter added 2 commits April 28, 2022 15:34

server: Remove GraphQLRequest

475b65b

This didn't need to be a future; just a synchronous helper function is enough

graph, server: Measure query timings for successful and failed queries

cadda88

Before, we weren't recording failed queries at all

lutter force-pushed the lutter/query-err branch from c7c4c76 to cadda88 Compare April 28, 2022 22:34

lutter merged commit cadda88 into master Apr 28, 2022

lutter deleted the lutter/query-err branch April 28, 2022 22:34

Provide feedback