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

Decouple post-processing from requests for Notifications #166

Open
uklotzde opened this issue Mar 6, 2019 · 1 comment
Open

Decouple post-processing from requests for Notifications #166

uklotzde opened this issue Mar 6, 2019 · 1 comment

Comments

@uklotzde
Copy link
Member

uklotzde commented Mar 6, 2019

Modifying requests (PUT/POST/DELETE) often require to execute post-processing actions like re-indexing some entries or sending subscription e-mails. Currently those actions are executed within the request context.

The request should succeed as soon as the actual operation has completed successfully. Required follow-up actions need to be scheduled within the database transaction, but must not be executed immediately -> event log table(s). The actual execution is performed later by independent tasks that are triggered on-demand and/or periodically.

@wellemut wellemut changed the title Decouple post-processing from requests Decouple post-processing from requests for Notifications Apr 20, 2020
@wellemut
Copy link
Member

If we build the subscription process completly on the "recently-changed"-API, there is not post precessing for notifications at all anymore:
grafik

Checkout: #258 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Review
Development

No branches or pull requests

4 participants