New IBootstrap registration for resource and room back ends #29563
Labels
3. to review
Waiting for reviews
bug
feature: caldav
Related to CalDAV internals
feature: dav
integration
performance 🚀
\OCP\Calendar\Resource\IManager::registerBackend
and\OCP\Calendar\Room\IManager::registerBackend
use an old style of service registration. You can't use them in\OCA\CalendarResourceManagement\AppInfo\Application::register
of an app because you have to fetch the container.This raises the next problem: the full service and its dependencies have to be built for every single request, despite resource being used in it or not. So this has a small but factual performance penalty on installations that have apps with resource back-ends.
I propose to move the registration to the registration context and fetch the registered backends from the registration context on demand. Then the registration itself becomes lightweight and we don't have to build services that won't be used for the majority of requests.
The text was updated successfully, but these errors were encountered: