Merge pull request #66 from michalc/refactor/simplify-finalize

refactor: remove dead code to simplify finalization

Author: michalc

sqlite_s3_query.py

@@ -295,11 +295,7 @@ def get_pp_stmt(statement):
                 raise Exception('Attempting to use finalized statement') from None
 
         def finalize(statement):
-            # In case there are errors, don't attempt to re-finalize the same statement
-            try:
-                pp_stmt = statements.pop(statement)
-            except KeyError:
-                return
+            pp_stmt = statements.pop(statement)
 
             try:
                 run_with_db(db, libsqlite3.sqlite3_finalize, pp_stmt)
@@ -323,11 +319,7 @@ def get_pp_stmts(sql):
                 statements[statement] = pp_stmt
                 yield partial(get_pp_stmt, statement), partial(finalize, statement)
 
-        try:
-            yield get_pp_stmts
-        finally:
-            for statement in statements.copy().keys():
-                finalize(statement)
+        yield get_pp_stmts
 
     def rows(get_pp_stmt, columns):
         while True: