Aklakan commented on issue #2233:
URL: https://github.com/apache/jena/issues/2233#issuecomment-1935079561

   The use case is to improve abstraction: Once the connection is configured 
with the send mode, then the rest of the application no longer has to concern 
itself with that aspect. Any call to `conn.query()` or `conn.newQuery()` will 
default to the configured send mode. This is useful when the triple store one 
connects to e.g. does not support `QuerySendMode.asGetWithLimitBody`. 
Otherwise, the application would have to break the encapsulation of the 
RDFConnection for each query: upcast the QueryExecBuilder to 
QueryExecHTTPBuilder and set the send mode - which wouldn't work for a 
RDFConnection to a Dataset.
   
   > There seem to be visibility changes that don't seem necessary.
   The constructor of `QueryExecHTTBuilder` could go from private to protected 
(rather than public). I think the other private attributes are passed to the 
`buildX` function, so their visibility could remain unchanged.
   
   Will check in detail and separate the PR tomorrow.


-- 
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