Github user ajs6f commented on a diff in the pull request:
https://github.com/apache/jena/pull/95#discussion_r45620222
--- Diff:
jena-fuseki2/jena-fuseki-core/src/main/java/org/apache/jena/fuseki/servlets/ResponseResultSet.java
---
@@ -309,13 +296,81 @@ private static void output(HttpAction action, String
contentType, String charset
out.println("## Query cancelled due to timeout during
execution ##") ;
out.println("## **** Incomplete results
**** ##") ;
out.flush() ;
- // No point raising an exception - 200 was sent already.
+ // No point raising an exception - 200 was sent already.
//errorOccurred(ex) ;
}
- // Includes client gone.
- } catch (IOException ex)
+ // Includes client gone.
+ } catch (IOException ex)
{ ServletOps.errorOccurred(ex) ; }
// Do not call httpResponse.flushBuffer(); here - Jetty closes the
stream if it is a gzip stream
- // then the JSON callback closing details can't be added.
+ // then the JSON callback closing details can't be added.
+ }
+
+ private static void output(HttpAction action, String contentType,
String charset, OutputContent proc, CacheAction cacheAction)
+ {
+ try {
+ setHttpResponse(action, contentType, charset) ;
+ action.response.setStatus(HttpSC.OK_200) ;
+
+ OutputStream outServlet = action.response.getOutputStream();
+ Cache cache = SPARQL_Query_Cache.getCache();
--- End diff --
Shouldn't this be properly parameterized?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
