Skip to content

Commit 3a2c8a8

Browse files
Merge pull request #110 from duneanalytics/fix_not_archiving_failed_queries
Introduce fix to archive failed queries on run_sql call
2 parents a87bf53 + 668142c commit 3a2c8a8

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

dune_client/api/extensions.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -253,11 +253,13 @@ def run_sql(
253253
Requires Plus subscription!
254254
"""
255255
query = self.create_query(name, query_sql, params, is_private)
256-
results = self.run_query(
257-
query=query.base, performance=performance, ping_frequency=ping_frequency
258-
)
259-
if archive_after:
260-
self.archive_query(query.base.query_id)
256+
try:
257+
results = self.run_query(
258+
query=query.base, performance=performance, ping_frequency=ping_frequency
259+
)
260+
finally:
261+
if archive_after:
262+
self.archive_query(query.base.query_id)
261263
return results
262264

263265
######################

0 commit comments

Comments
 (0)