mirror of
https://github.com/hasura/graphql-engine.git
synced 2024-12-17 20:41:49 +03:00
512a4dbb92
### Description This PR changes all the schema code to operate in a specific `SchemaT` monad, rather than in an arbitrary `m` monad. `SchemaT` is intended to be used opaquely with `runSourceSchema` and `runRemoteSchema`. The main goal of this is to allow a different reader context per part of the schema: this PR also minimizes the contexts. This means that we no longer require `SchemaOptions` when building remote schemas' schema, and this PR therefore removes a lot of dummy / placeholder values accordingly. ### Performance and stacking This PR has been through several iterations. #5339 was the original version, that accomplished the same thing by stacking readers on top of the stack at every remote relationship boundary. This raised performance concerns, and @0x777 confirmed with an ad-hoc test that in some extreme cases we could see up to a 10% performance impact. This version, while more verbose, allows us to unstack / re-stack the readers, and avoid that problem. #5517 adds a new benchmark set to be able to automatically measure this on every PR. ### Remaining work - [x] a comment (or perhaps even a Note?) should be added to `SchemaT` - [x] we probably want for #5517 to be merged first so that we can confirm the lack of performance penalty PR-URL: https://github.com/hasura/graphql-engine-mono/pull/5458 GitOrigin-RevId: e06b83d90da475f745b838f1fd8f8b4d9d3f4b10 |
||
|---|---|---|
| .. | ||
| BoolExp | ||
| Action.hs | ||
| Backend.hs | ||
| BoolExp.hs | ||
| Build.hs | ||
| Common.hs | ||
| Instances.hs | ||
| Introspect.hs | ||
| Mutation.hs | ||
| NamingCase.hs | ||
| Node.hs | ||
| Options.hs | ||
| OrderBy.hs | ||
| Parser.hs | ||
| Postgres.hs | ||
| Relay.hs | ||
| Remote.hs | ||
| RemoteRelationship.hs | ||
| Select.hs | ||
| SubscriptionStream.hs | ||
| Table.hs | ||
| Typename.hs | ||
| Update.hs | ||