Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Parsed Persisted queries caching #919

Closed
Sytten opened this issue May 9, 2022 · 1 comment
Closed

Parsed Persisted queries caching #919

Sytten opened this issue May 9, 2022 · 1 comment
Labels
question Further information is requested

Comments

@Sytten
Copy link

Sytten commented May 9, 2022

Hello!
I saw the extension for the APQ that is based on strings and I was wondering if there is something prevent us from building a similar extension but to cache the parsed struct (ExecutableDocument?).
In terms of performances it would be much better to not have to reparse the query each time and just parse the variables.
Thanks!
@Sytten Sytten added the question Further information is requested label May 9, 2022
@sunli829
Copy link
Collaborator

Good idea, I'll do it now! 😁

sunli829 added a commit that referenced this issue May 13, 2022
@sunli829
Cache parsed ExecuteDocument in APQ. #919
440fe98
@Sytten Sytten closed this as completed May 24, 2022
@rieval rieval mentioned this issue Nov 9, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sytten @sunli829