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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to