Muster zur Performancesteigerung der GraphQL Anfragen (SaaS)

[marcometz]

In einer Query können beliebige "nested resources" mit abgefragt werden. Diese werden bisher jedoch im ActiveRecord scope nicht mit includes berücksichtigt, sodass für jeden Treffer zusätzliche SQL anfragen notwendig werden.

Pauschal immer alle möglichen Ressourcen zusätzlich zu laden ist jedoch auch keine Lösung.

Passend dafür wäre es also gut schon beim scope zu wissen welche Ressourcen zusätzlich benötigt werden. Hierfür kann "extras: [:lookahead]" verwendet werden. dies gibt dem Context zusätzliches Wissen mit, welches im Scope verwendet werden kann.

Beispielsweise wie hier um passende :includes an den scope ketten.

Durch das hinzuladen der weiteren Models sind dann weitere Besonderheiten zu beachten, wie hier zum Beispiel die klare Benennung der Sortierargumente, da diese nun nicht mehr eindeutig sind.

https://graphql-ruby.org/queries/lookahead.html
https://evilmartians.com/chronicles/how-to-graphql-with-ruby-rails-active-record-and-no-n-plus-one

context[:current_arguments][:lookahead].selections.map(&:name) hat in diesem Beispiel dann folgende Werte:

[:id, :title, :list_date, :data_provider, :dates, :addresses, :categories]

[codeclimate]

Code Climate has analyzed commit 77af414 and detected 5 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Style	5

View more on Code Climate.

[donni106]

This is going to staging now with the implemented update for event records for a testing stage.

Edit: FAILED!

undefined method `extras' for Resolvers::EventRecordsSearch:Class

I have reset staging and put this issue aside.

[marcometz]

Habe es erneut getestet. Das ganze funktioniert nur mit einer neueren graphql Version, wie sie im SAAS-Branch verwendet wird.

Ein Upgrade des graphQL Gems alleine reicht jedoch nicht, weil es sehr viele weitere notwendige Anpassungen gibt, damit die neue Version auf Master wieder funktioniert. (siehe PR Saas<->Master auf Gitlab)