-
-
Notifications
You must be signed in to change notification settings - Fork 218
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
Performance ESCR tests. Disabling Neos.Neos catchUpHooks
speeds up tests
#4878
Labels
Comments
catchUpHooks
speeds testscatchUpHooks
speeds up tests
Fine for me, the question is at what point the tests are no longer representative of the real world. Does the hook have dedicated tests for example? |
mhsdesign
added a commit
to mhsdesign/neos-development-collection
that referenced
this issue
Feb 20, 2024
Resolves: neos#4878 by setting it to null or `~` That for example allows to speed up local e2e tests by diabling certain neos specific projections: ```yaml Neos: ContentRepositoryRegistry: presets: 'default': projections: 'Neos.Neos:DocumentUriPathProjection': catchUpHooks: 'Neos.Neos:FlushRouteCache': ~ 'Neos.Neos:PendingChangesProjection': ~ 'Neos.ContentRepository:ContentGraph': catchUpHooks: 'Neos.Neos:FlushContentCache': ~ 'Neos.Neos:AssetUsage': ~ ```
6 tasks
With #4904 i allow a workaround to do this locally which seems fine for now? Neos:
ContentRepositoryRegistry:
presets:
'default':
projections:
'Neos.Neos:DocumentUriPathProjection':
catchUpHooks:
'Neos.Neos:FlushRouteCache': ~
'Neos.Neos:PendingChangesProjection': ~
'Neos.ContentRepository:ContentGraph':
catchUpHooks:
'Neos.Neos:FlushContentCache': ~
'Neos.Neos:AssetUsage': ~ do you agree? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Related: #4750
running
takes 28 seconds currently.
When disabling the unneeded (for those tests)
RouterCacheHookFactory
andGraphProjectorCatchUpHookForCacheFlushingFactory
neos hooks it improves to 21 seconds.And also excluding the
PendingChangesProjection
will make it to about 19 seconds.The text was updated successfully, but these errors were encountered: