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

Update the query to only return the count from the table since that is

Browse Source 
all we care about.

https://www.pivotaltracker.com/story/show/180262357
This commit is contained in:
Rebecca Law
2021-11-17 14:46:52 +00:00
parent bffca39223
commit 30a5852685
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 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) 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(insert_query)
db.session.execute(delete_query) db.session.execute(delete_query)
db.session.execute("DROP TABLE NOTIFICATION_ARCHIVE") 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): def _move_notifications_to_notification_history(notification_type, service_id, day_to_delete_backwards_from, qry_limit):