You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Correct me, if I'm wrong, It seems that graphQLSchema and graphqlFieldVisibility are set once when GraphQLJpaExecutorContext is created, and the result of transformation is always the same, therefore this transformation can be applied only once when GraphQLJpaExecutorContext is created and not on every request. When we updated this code, we got a huge performance boost.
BTW, thanks for library, it's cool!
The text was updated successfully, but these errors were encountered:
Hi,
GraphQLJpaExecutorContext
transforms the schema field visibility rules on every request and it leads to performance drop.Specifically, in https://github.com/introproventures/graphql-jpa-query/blob/master/graphql-jpa-query-schema/src/main/java/com/introproventures/graphql/jpa/query/schema/impl/GraphQLJpaExecutorContext.java#L108 :
Correct me, if I'm wrong, It seems that
graphQLSchema
andgraphqlFieldVisibility
are set once whenGraphQLJpaExecutorContext
is created, and the result of transformation is always the same, therefore this transformation can be applied only once whenGraphQLJpaExecutorContext
is created and not on every request. When we updated this code, we got a huge performance boost.BTW, thanks for library, it's cool!
The text was updated successfully, but these errors were encountered: