Catch more exceptions to clean up task store

The failure conditions investigated in the wild (AuraDS) were very hard
to analyse conclusively. This is an attempt at being more comprehensive
in our cleaning up of the task store.

Co-authored-by: Florentin Dörre <florentin.dorre@neo4j.com>

Author: Mats-SX

legacy-cypher-projection/src/main/java/org/neo4j/gds/legacycypherprojection/CypherFactory.java

@@ -162,8 +162,8 @@ public GraphDimensions estimationDimensions() {
     @Override
     public CSRGraphStore build() {
         // Temporarily override the security context to enforce read-only access during load
-        return readOnlyTransaction().apply((tx, ktx) -> {
-            try {
+        try {
+            return readOnlyTransaction().apply((tx, ktx) -> {
                 progressTracker.beginSubTask("Loading");
                 BatchLoadResult nodeCount = new CountingCypherRecordLoader(
                     cypherConfig.nodeQuery(),
@@ -198,11 +198,11 @@ public CSRGraphStore build() {
                 logLoadingSummary(graphStore);
 
                 return graphStore;
-            } catch (RuntimeException ex) {
-                progressTracker.endSubTaskWithFailure();
-                throw ex;
-            }
-        });
+            });
+        } catch (Throwable ex) {
+            progressTracker.endSubTaskWithFailure();
+            throw ex;
+        }
     }
 
     private ProgressTracker initProgressTracker() {