Re: [GitHub] jena issue #236: No stacktrace on regex errors. Cancel/close during sorting.

[Andy Seaborne]

On 11/04/17 15:49, Chris Dollin wrote:
diff --git
a/jena-arq/src/main/java/org/apache/jena/sparql/engine/iterator/QueryIterSort.java
b/jena-arq/src/main/java/org/apache/jena/sparql/engine/iterator/QueryIterSort.java
index 173da34..04c2655 100644
---
a/jena-arq/src/main/java/org/apache/jena/sparql/engine/iterator/QueryIterSort.java
+++
b/jena-arq/src/main/java/org/apache/jena/sparql/engine/iterator/QueryIterSort.java
@@ -92,6 +92,7 @@ public class QueryIterSort extends QueryIterPlainWrapper {
             // iterator in a try/finally block, and thus will call
             // close() themselves.
             catch (QueryCancelledException e) {
+               QueryIterSort.this.close();

Is this needed? The query is cancelled so QueryIterSort.requestCancel 
happens?




                 close();
                 throw e;
             }