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.

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

Processor being cached for each connection instead recreate #240

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nikitakuhta
Copy link

For every query processor creates and make deepClone of whole schema. It can take much time (100-300 ms with 300 models). Because of this, we have huge overhead in each query (query takes 1-5ms, deepClone takes 150ms). Very big problem is, that deepClone works synchronously, and whole application can be frozen, when many queries being executed.
In this pull request processor being cached in connection object, and no need to recreate it every query and make resource-expensive deepClone. Schema have no modifications in here, it don't need to be recreated every time.

@nikitakuhta nikitakuhta changed the title Adapter cache processor for each connection instead recreate Processor being cached for each connection instead recreate Apr 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant