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

Spiral: Add Cycle ORM config #9034

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

roxblnfk
Copy link

@roxblnfk roxblnfk commented May 17, 2024

What was changed:

Added Cycle ORM config with cache=true

Why?

#9021 (comment)

I suppose that unnecessary requests arise because each of the RoadRunner workers synchronizes the ORM schema with the database.
Enabling cache might help avoid the issue.

@joanhey
Copy link
Contributor

joanhey commented May 17, 2024

The problem with the update test is necessary to fix it to pass the tests.
The ORM for the number of queries need to update that number of rows in the database.
Well forcing to update all the worlds and/or forcing to have different random number for ID.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants