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

Andy Seaborne commented on JENA-65:
-----------------------------------

Looks reasonable though "new Quad(g,s,p,o)" is very cheap.

It'll still need Quads because of find().

Could you give some more background information as to the Quadruple abstraction 
(mailing list preferably - it's specifically about this JIRA)?  Be helpful to 
understand context for when we change internal abstractions in any next major 
version.



> Provide add and remove methods with nodes as paramaters in DatasetGraph 
> ------------------------------------------------------------------------
>
>                 Key: JENA-65
>                 URL: https://issues.apache.org/jira/browse/JENA-65
>             Project: Jena
>          Issue Type: Improvement
>          Components: ARQ
>            Reporter: Laurent Pellegrino
>            Assignee: Andy Seaborne
>              Labels: api, arq
>         Attachments: arq-add-delete.patch
>
>
> From an instance of DatasetGraphTDB for example, it is possible to add or to 
> remove quads with a Quad as parameter. However if we have the set of nodes 
> associated to a Quad (g, s, p, o), it is not possible to directly add or 
> delete a quad by using these nodes. In order to do that we have to create a 
> new Quad with the nodes. I think it would be nice to provide an add(Node g, 
> Node s, Node p, Node o) and a delete(Node g, Node s, Node p, Node o). First, 
> to be consistent because you already provide it for contains(Node g, Node s, 
> Node p, Node o), find(Node g, Node s, Node p, Node o) and deleteAny(Node g, 
> Node s, Node p, Node o). But also because it offers the possibility to avoid 
> to create Quads for people who provide their own Quadruple abstraction (which 
> uses Jena Nodes).

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to