apollographql / federation-next

Moved to the Router repository
https://github.com/apollographql/router/tree/dev/apollo-federation
Other
19 stars 1 forks source link

Add a temporary error code definition for internal errors #274

Closed lrlna closed 4 months ago

lrlna commented 4 months ago

We are using internal errors a ton inside the query planner, and while ErrorCode::definition is not yet used, I don't want to accidentally have us run into a panic with error codes. There is a follow up ticket for documenting this when we deal with composition later on - https://apollographql.atlassian.net/browse/FED-190

lrlna commented 4 months ago

I am pretty sure it's not in the hot path, but there are like 20 internal errors in the query planner 🙈 so just in case