-
-
Notifications
You must be signed in to change notification settings - Fork 223
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
Improve performance for publish/discard of workspaces by using the synchronous mode #4285
Comments
... this ensures that during publishing of nodes, we do not fork sub-processes for event processing (for every event). This speeds up the system usually by at least factor 2-3 during these operations. It *could* have some side-effects if caches were not properly cleared, but at least I did not see any yet (and we run tests often with this flag). Resolves: neos/neos-development-collection#4285
... this ensures that during publishing of nodes, we do not fork sub-processes for event processing (for every event). This speeds up the system usually by at least factor 2-3 during these operations. It *could* have some side-effects if caches were not properly cleared, but at least I did not see any yet (and we run tests often with this flag). Resolves: #4285
... this ensures that during publishing of nodes, we do not fork sub-processes for event processing (for every event). This speeds up the system usually by at least factor 2-3 during these operations. It *could* have some side-effects if caches were not properly cleared, but at least I did not see any yet (and we run tests often with this flag). Resolves: neos/neos-development-collection#4285
For testing i enabled the synchronous mode for publishing Nodes in the Neos.Ui and yes the performance will be drastically improved. Maybe by only blocking for the content graph we can speed up the process while still maintaining our philosophy? (as suggested by @bwaidelich) $cr->handle($command)->block(ContentGraphProjection::class);
// next code that requires the ContentGraphProjection to be up to date but doest "care" about other projections. Related: #4746 I think releasing Neos 9 without any further performance improvements - especially if the are so easy and only our own concepts are holding us back - would be a mistake. |
This will be obsolete with #4746 |
During publishing of nodes, we usually fork sub-processes for event processing (for every event). This drastically slows down these operations. We can enable synchronous mode in these methods, to speed up the system at least by a factor 2-3 for these operations.
It could have some side-effects if caches were not properly cleared,
but at least I did not see any yet (and we run tests often with this flag).
Related: #4388
The text was updated successfully, but these errors were encountered: