Defect #627
closed
Refactor query for email notification (recipient)
100%
Description
Please refactor query for get emails and add indexes fro realRecipient.
Use case: When search email notifications and fill recipient, query for search is very slow (~5 minutes).
Files
Updated by Ondřej Kopr over 6 years ago
- Target version changed from Diamond (7.4.0) to Citrine (7.3.0)
After discussion with implementator's team, change target version to Citrine.
Updated by Marcel Poul over 6 years ago
the result of the filter is also invalid. see previous url and usecase when I use user mpoul. The result I get consists of ALL sent emails (12721). The same number as with cleaned filter.
Updated by Radek Tomiška over 6 years ago
- Status changed from New to In Progress
Updated by Radek Tomiška over 6 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Ondřej Kopr
- % Done changed from 0 to 90
All repositories are using the same criteria api in AbstractNotificationLogService#toPredicates method. Query is much simpler now (exists parts is not needed now):
https://github.com/bcvsolutions/CzechIdMng/commit/7e305bdc251de8a085df88736adb57ae06b3b524
Could you do a feedback please?
Updated by Ondřej Kopr over 6 years ago
- File email_test.png email_test.png added
- File email_time.png email_time.png added
- Status changed from Needs feedback to Resolved
- Assignee changed from Ondřej Kopr to Radek Tomiška
- % Done changed from 90 to 100
Awesome! I did test with ~13000 rows (same as projects) and query time is about 400ms! Thanks for refactor this query. Thanks.
Updated by Radek Tomiška over 6 years ago
- Status changed from Resolved to Closed