Aklakan commented on PR #3511:
URL: https://github.com/apache/jena/pull/3511#issuecomment-4103458726

   > Is QueryExecApp still needed?
   
   Unless I am missing something, QueryExecApp should be unaffected by the 
builder indirection:
   
   * `QueryExecBuilderDeferred` is a QueryExec_Builder_ view defers creation of 
a DatasetGraph-specific builder.
   * `QueryExecApp` is a QueryExec view with support for mutable timeout 
properties via the `QueryExecMod` interface. `QueryExecApp` is backed by a 
(possibly deferred) builder such that timeouts can still be set.
   IIRC, mutable timeout settings on the QueryExecution is legacy from at least 
late jena 3.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to