darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-31 12:51:47 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #3376 from alphagov/update-query-for_insert_notification_history_delete_notifications

Browse Source 
Small update to query to reduce load on the task.
This commit is contained in:
Rebecca Law
2021-11-18 11:28:44 +00:00
committed by GitHub
parent bffca39223 30a5852685
commit 443f197fee
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/dao/notifications_dao.py
View File

@@ -398,14 +398,14 @@ def insert_notification_history_delete_notifications(
select_to_use = select_into_temp_table_for_letters if notification_type == 'letter' else select_into_temp_table
db.session.execute(select_to_use, input_params)
result = db.session.execute("select * from NOTIFICATION_ARCHIVE")
result = db.session.execute("select count(*) from NOTIFICATION_ARCHIVE").fetchone()[0]
db.session.execute(insert_query)
db.session.execute(delete_query)
db.session.execute("DROP TABLE NOTIFICATION_ARCHIVE")
return result.rowcount
return result
def _move_notifications_to_notification_history(notification_type, service_id, day_to_delete_backwards_from, qry_limit):