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

Merge pull request #2770 from alphagov/increase-qry-limit

Browse Source 
Increase default query limit to 50,000 from 10,000
This commit is contained in:
Rebecca Law
2020-03-31 17:15:12 +01:00
committed by GitHub
parent a7a158fe69 e5b6fa2143
commit ac8fc49539
1 changed files with 2 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
app/dao/notifications_dao.py
View File

@@ -297,7 +297,7 @@ def _filter_query(query, filter_dict=None):
@statsd(namespace="dao")
def delete_notifications_older_than_retention_by_type(notification_type, qry_limit=10000):
def delete_notifications_older_than_retention_by_type(notification_type, qry_limit=50000):
current_app.logger.info(
'Deleting {} notifications for services with flexible data retention'.format(notification_type))
@@ -371,19 +371,14 @@ def insert_notification_history_delete_notifications(
"timestamp_to_delete_backwards_from": timestamp_to_delete_backwards_from,
"qry_limit": qry_limit
}
current_app.logger.info(
f"Start executing insert_notification_history_delete_notifications for input params {input_params}"
)
db.session.execute(drop_table_if_exists)
current_app.logger.info('Start executing select into temp table')
db.session.execute(select_into_temp_table, input_params)
result = db.session.execute("select * from NOTIFICATION_ARCHIVE")
current_app.logger.info('Start executing insert into history')
db.session.execute(insert_query)
current_app.logger.info('Start executing deleting notifications')
db.session.execute(delete_query)
db.session.execute("DROP TABLE NOTIFICATION_ARCHIVE")