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

The notifications table should have an index on "recipient" #5454

Closed
andreas-gruenwald opened this issue Dec 16, 2019 · 1 comment
Closed

The notifications table should have an index on "recipient" #5454

andreas-gruenwald opened this issue Dec 16, 2019 · 1 comment

Comments

@andreas-gruenwald
Copy link
Contributor

@andreas-gruenwald andreas-gruenwald commented Dec 16, 2019

Feature Request

The notification table is polled frequently in the Pimcore backend.

  1. Many applications event don't use this feature. Is there a way to disable notifications?
  2. ALTER TABLE `notifications` ADD INDEX `recipient` (`recipient`); should be applied to the database table so that querying notifications for a specific Pimcore user is done via index.
@brusch

This comment has been minimized.

Copy link
Member

@brusch brusch commented Dec 18, 2019

Currently there's no way to disable the feature globally, I'd say that's a topic for another issue.
The index on the notifications table should be added to the install.sql + a migration script.

@brusch brusch added this to To do in #76 | 07/01/2020 - 17/01/2020 via automation Jan 7, 2020
@dvesh3 dvesh3 closed this in 6fff423 Jan 16, 2020
#76 | 07/01/2020 - 17/01/2020 automation moved this from To do to Done Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.