[ 
https://issues.apache.org/jira/browse/JENA-1389?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16152841#comment-16152841
 ] 

A. Soroka edited comment on JENA-1389 at 9/4/17 6:16 PM:
---------------------------------------------------------

This seems quite reasonable to me, because it wouldn't break any current 
clients of `Dataset`. But it would break current implementations. Since 
`DatasetGraph`, not `Dataset`, is the SPI, that doesn't bother me, but how do 
other people feel?


was (Author: ajs6f):
This seems quite reasonable to me.

> Return `this` rather than `void` from Dataset
> ---------------------------------------------
>
>                 Key: JENA-1389
>                 URL: https://issues.apache.org/jira/browse/JENA-1389
>             Project: Apache Jena
>          Issue Type: Improvement
>          Components: ARQ
>    Affects Versions: Jena 3.4.0
>            Reporter: Adam Jacobs
>            Priority: Trivial
>              Labels: easytask
>
> Allow method chaining from the org.apache.jena.query.Dataset interface by 
> returning `this` rather than `void` from the following methods.
> # setDefaultModel
> # addNamedModel
> # removeNamedModel
> # replaceNamedModel
> Allowing method chaining would align with the behavior of the add and remove 
> methods in org.apache.jena.rdf.model.Model.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to